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

Migrator Default Timestamps #852

Merged
merged 1 commit into from Apr 24, 2018
Merged

Conversation

@charlycontreras
Copy link
Contributor

charlycontreras commented Apr 24, 2018

  • Migrator now automatically manages the timestamp columns on addRecord() and updateRecord() calls.
  • Migrator correctly honors CFWheels Timestamp configuration settings (setUpdatedAtOnCreate, softDeleteProperty, timeStampMode, timeStampOnCreateProperty, timeStampOnUpdateProperty) .
* Migrator now automatically manages the timestamp columns on addRecord() and updateRecord() calls.
* Migrator correctly honors CFWheels Timestamp configuration settings (setUpdatedAtOnCreate, softDeleteProperty, timeStampMode, timeStampOnCreateProperty, timeStampOnUpdateProperty) .
@perdjurner perdjurner merged commit d0ce89e into cfwheels:master Apr 24, 2018
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@perdjurner

This comment has been minimized.

Copy link
Contributor

perdjurner commented Apr 24, 2018

Thank you!

@chapmandu

This comment has been minimized.

Copy link
Member

chapmandu commented Apr 24, 2018

Nice work!

@neokoenig

This comment has been minimized.

Copy link
Member

neokoenig commented Apr 24, 2018

Great stuff!

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

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.