commanded-ecto-projections icon indicating copy to clipboard operation
commanded-ecto-projections copied to clipboard

Use `timestamp with time zone` for timestamp fields in `projection_versions`

Open datafoo opened this issue 6 years ago • 2 comments

The timestamps inserted_at and updated_at in projection_versions table use the PostgreSQL type timestamp without time zone.

Since inserted_at and updated_at are to be implicitly interpreted as UTC, why not making it explicit and use the PostgreSQL type timestamp with time zone instead?

datafoo avatar Jan 25 '19 14:01 datafoo

I'm unsure why I chose to use timestamp without timezone for these fields, possibly because I thought NaiveDateTime was more appropriate than DateTime.

Am happy to accept a pull request to change these.

slashdotdash avatar Jan 25 '19 17:01 slashdotdash

I may work on this pull request.

datafoo avatar Jan 25 '19 17:01 datafoo