Skip to content

Add Timestamped base model (created_at, last_updated_at auto fields). - #3023

Open
arkid15r wants to merge 1 commit into
devfrom
arkid15r/3019-unify-model-timestamps
Open

Add Timestamped base model (created_at, last_updated_at auto fields).#3023
arkid15r wants to merge 1 commit into
devfrom
arkid15r/3019-unify-model-timestamps

Conversation

@arkid15r

@arkid15r arkid15r commented May 6, 2022

Copy link
Copy Markdown
Contributor

Issue Number

#3019
Introduce Timestamped model base with created_at, last_modified_at auto fields

@davidsmejia

Copy link
Copy Markdown
Contributor

I'd much prefer we kept created_at and updated_at conventions across models.

@davidsmejia davidsmejia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's update the field name and make a ticket to verify that the new fields don't get auto inserted, I am concerned about there being a timeout, might make sense to add the field and then paginate and insert the values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants