Skip to content

Track last update#2204

Merged
jesseleite merged 10 commits intomasterfrom
feature/track-last-update
Aug 13, 2020
Merged

Track last update#2204
jesseleite merged 10 commits intomasterfrom
feature/track-last-update

Conversation

@jesseleite
Copy link
Contributor

@jesseleite jesseleite commented Aug 11, 2020

  • Add new updateLastModified() functionality to entries and terms.
  • Add new touch() method similar to Eloquent's.
  • Add new track_last_update config option to disable updated_at and updated_by tracking. Closes Ability to disable "updated_by" on save #1339.

@jesseleite jesseleite requested a review from jasonvarga August 11, 2020 21:02
Copy link
Member

@jasonvarga jasonvarga left a comment

Choose a reason for hiding this comment

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

Can we move the config option into users.phpsystem.php?

This is feature isn't really cp specific.

You can call updateLastModified() from anywhere, even outside the CP (for instance, in Workshop down the road)

Copy link
Member

@jasonvarga jasonvarga left a comment

Choose a reason for hiding this comment

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

Also, let's require a user be passed into updateLastModified rather than fall back to the auth'd user.

No user passed in, updated_by gets nulled out.

@jesseleite jesseleite merged commit bf07fe6 into master Aug 13, 2020
@jesseleite jesseleite deleted the feature/track-last-update branch August 13, 2020 21:03
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.

Ability to disable "updated_by" on save

2 participants