Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creating timestamp fields instead of time type for postgres #687

Open
Tracked by #138
jwaldner opened this issue Feb 3, 2022 · 0 comments
Open
Tracked by #138

Creating timestamp fields instead of time type for postgres #687

jwaldner opened this issue Feb 3, 2022 · 0 comments
Labels
s: triage Some tests need to be run to confirm the issue

Comments

@jwaldner
Copy link

jwaldner commented Feb 3, 2022

Hello! Thank you so much for your interest in Pop. The fact you care enough to be here, right now, helping makes us very happy.

Delete the above section and the instructions in the sections below before submitting.

Description

this is the migration up

create_table("children") {
t.Column("id","integer", {primary: true})
t.Column("first_name", "string", {})
t.Column("last_name", "string", {})
t.Column("birth_date", "timestamp", { })
t.Column("image", "blob", {})
}

create_table("check_ins") {
t.Column("child_id", "int")
t.Column("checkin_id", "date")
t.PrimaryKey("child_id", "checkin_id")

t.Column("first_in", "time", {})
t.Column("first_out", "time", {})

}

Steps to Reproduce the Problem

migration up

Expected Behavior

time fields for first_in & first_out
What did you what to happen? Tell us a story. We love to read.

Actual Behavior

timestamp & timestampz
In the happiest of happy places what should have happened?

Info

pop 6.0.1

Please precise your OS, the Pop version and if you're using Pop through Buffalo.

Just now found a work around, use "time " in the field type instead of "time"

@sio4 sio4 added the s: triage Some tests need to be run to confirm the issue label Sep 20, 2022
@sio4 sio4 added this to the Backlog milestone Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s: triage Some tests need to be run to confirm the issue
Projects
None yet
Development

No branches or pull requests

2 participants