Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

deposit: fix log date format #335

Merged
merged 1 commit into from
Oct 21, 2020
Merged

deposit: fix log date format #335

merged 1 commit into from
Oct 21, 2020

Conversation

sebdeleze
Copy link
Contributor

Co-Authored-by: Sébastien Délèze sebastien.deleze@rero.ch

@sebdeleze sebdeleze marked this pull request as ready for review October 6, 2020 06:41
@sebdeleze sebdeleze requested a review from zannkukai October 6, 2020 06:41
@@ -84,7 +86,7 @@ def log_action(self, user, action, comment=None):

log = {
'user': user,
'date': datetime.now().strftime('%Y-%m-%d %H:%M:%S'),
'date': pytz.utc.localize(datetime.utcnow()).isoformat(),

Choose a reason for hiding this comment

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

👍

* Fixes date format to UTC timezone for deposit logs.
* Closes #333.

Co-Authored-by: Sébastien Délèze <sebastien.deleze@rero.ch>
@pronguen pronguen self-requested a review October 21, 2020 12:34
@sebdeleze sebdeleze merged commit a19021f into rero:dev Oct 21, 2020
@sebdeleze sebdeleze deleted the sed-fix-deposit-log-date branch October 21, 2020 14:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Time zone is wrong for dates in the logs of deposits.
3 participants