Skip to content

fix: only create new check in IDs on "start" - #1252

Merged
whatyouhide merged 2 commits into
masterfrom
al/check-in-id
Oct 23, 2026
Merged

whatyouhide merged 2 commits into
masterfrom
al/check-in-id

Conversation

@whatyouhide

Copy link
Copy Markdown
Collaborator

Description

As per the title!

@solnic

solnic commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

@whatyouhide hey hey thank you, could you rebase onto main? We had a small CI hiccup and it's addressed there, it will make CI run over here.

@whatyouhide

Copy link
Copy Markdown
Collaborator Author

@solnic done

@solnic solnic left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved with a comment!

Comment thread test/sentry/integrations/oban/cron_test.exs Outdated
Oban and Quantum cron tests each defined the same expire_check_in_mapping!/1.
Move it into Sentry.TestHelpers and call that from both suites.

Co-Authored-By: Grok 4.7 <noreply@anthropic.com>
@whatyouhide
whatyouhide merged commit aa21537 into master Oct 23, 2026
14 checks passed
@whatyouhide
whatyouhide deleted the al/check-in-id branch October 23, 2026 14:39
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