Skip to content

Always set updated_at when upserting periodic jobs - #1045

Merged
brandur merged 1 commit into
masterfrom
brandur-set-periodic-job-updated-at
Sep 27, 2025
Merged

Always set updated_at when upserting periodic jobs#1045
brandur merged 1 commit into
masterfrom
brandur-set-periodic-job-updated-at

Conversation

@brandur

@brandur brandur commented Sep 27, 2025

Copy link
Copy Markdown
Contributor

Make sure to set updated_at when upserting periodic jobs on calls to
the pilot's PeriodicJobUpsert. Not having updated_at was resulting
in pilots with database backing to store zero values into their rows.

@brandur
brandur requested a review from bgentry September 27, 2025 05:31

@bgentry bgentry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah, was gonna fix this on the pro query side but this is simpler!

Comment thread CHANGELOG.md Outdated
Make sure to set `updated_at` when upserting periodic jobs on calls to
the pilot's `PeriodicJobUpsert`. Not having `updated_at` was resulting
in pilots with database backing to store zero values into their rows.
@brandur
brandur force-pushed the brandur-set-periodic-job-updated-at branch from fa03453 to eb8a48b Compare September 27, 2025 15:59
@brandur
brandur merged commit 7c2200a into master Sep 27, 2025
19 of 20 checks passed
@brandur
brandur deleted the brandur-set-periodic-job-updated-at branch September 27, 2025 16:06
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.

2 participants