Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: update template updated_at value #2729

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
Merged
merged 2 commits into from
Jun 30, 2022
Merged

fix: update template updated_at value #2729

merged 2 commits into from
Jun 30, 2022

Conversation

AbhineetJain
Copy link
Contributor

@AbhineetJain AbhineetJain commented Jun 29, 2022 •
edited
Loading

This PR fixes the template update time not reflecting on updating the template version.

Subtasks

  • change the query that updates the active version to set updated_at.
  • change the query that marks the template as deleted to set updated_at.
  • pass in current UTC Go time as updated_at when calling these queries.
  • update instances of updated_at using SQL now() time.

Fixes #2720

Demo

template_update_demo.mov

@AbhineetJain AbhineetJain force-pushed the abhineetjain/2720-template-updated-at branch from 048f0d1 to 4220080 Compare June 29, 2022 23:53
@AbhineetJain AbhineetJain requested a review from a team June 30, 2022 00:07
@AbhineetJain AbhineetJain enabled auto-merge (squash) June 30, 2022 12:04
@AbhineetJain AbhineetJain merged commit 9df6bc7 into main Jun 30, 2022
@AbhineetJain AbhineetJain deleted the abhineetjain/2720-template-updated-at branch June 30, 2022 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johnstcn johnstcn johnstcn approved these changes

@kylecarbs kylecarbs kylecarbs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

UI: templates LAST UPDATED field does not show value properly
3 participants
@AbhineetJain @johnstcn @kylecarbs