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.

Sign up for GitHub

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

Jump to bottom

Set version author to the author that restores a version and update date #1640

Closed
ulliholtgrave opened this issue Aug 17, 2022 · 1 comment · Fixed by #1675
Closed

Set version author to the author that restores a version and update date #1640

ulliholtgrave opened this issue Aug 17, 2022 · 1 comment · Fixed by #1675
Assignees
@timobrembeck
Labels
💡 feature New feature or request ⁉️ prio: low Not urgent, can be resolved in the distant future. ☺️ effort: low Should be doable in <4h
Milestone
22Q3

Comments

@ulliholtgrave
Copy link
Member

Motivation

If a user restores an old version the date and the author gets set to the old values. These should be updated as they are creating confusion and don't allow the user to track the changes to the versions. The restoration process also leads to a strange behavior of the dates and we have newer versions that have an old time stamp.
@ulliholtgrave ulliholtgrave added 💡 feature New feature or request ⁉️ prio: low Not urgent, can be resolved in the distant future. ☺️ effort: low Should be doable in <4h labels Aug 17, 2022
@ulliholtgrave ulliholtgrave added this to the 22Q4 milestone Aug 17, 2022
@ulliholtgrave ulliholtgrave changed the title Set version author to the author that restores a version Set version author to the author that restores a version and update date Aug 17, 2022
@timobrembeck timobrembeck self-assigned this Sep 9, 2022
@timobrembeck
Copy link
Member

The restoration process also leads to a strange behavior of the dates and we have newer versions that have an old time stamp.

I cannot reproduce this behavior. When I restore old versions, the date gets changed to the current timestamp.

@timobrembeck timobrembeck mentioned this issue Sep 9, 2022
Merged
@timobrembeck timobrembeck mentioned this issue Sep 10, 2022
Merged
@timobrembeck timobrembeck modified the milestones: 22Q4, 22Q3 Sep 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timobrembeck timobrembeck

Labels
💡 feature New feature or request ⁉️ prio: low Not urgent, can be resolved in the distant future. ☺️ effort: low Should be doable in <4h
Projects
None yet
Milestone
22Q3
Development

Successfully merging a pull request may close this issue.

Improve revision view digitalfabrik/integreat-cms
2 participants
@ulliholtgrave @timobrembeck