Skip to content

feat: Support latest identifier on checkins#42819

Merged
dcramer merged 1 commit into
masterfrom
feat/checkin-latest-api
Jan 5, 2023
Merged

feat: Support latest identifier on checkins#42819
dcramer merged 1 commit into
masterfrom
feat/checkin-latest-api

Conversation

@dcramer

@dcramer dcramer commented Jan 5, 2023

Copy link
Copy Markdown
Member

Add the ability to target latest as the checkin_id parameter, which will pull the most recent (creation date) check-in that is not yet marked as finished.

Fixes GH-42816

@dcramer
dcramer requested a review from a team January 5, 2023 18:08
@dcramer
dcramer requested a review from a team as a code owner January 5, 2023 18:08
@dcramer
dcramer requested a review from davidenwang January 5, 2023 18:08
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 5, 2023
Comment thread src/sentry/api/base.py Outdated

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

request is always present here so no need to use our internal abstraction

@dcramer

This comment was marked as resolved.

Comment thread src/sentry/api/base.py Outdated
Add the ability to target `latest` as the `checkin_id` parameter, which
will pull the most recent (creation date) check-in that is not yet marked
as finished.

Fixes GH-42816

@davidenwang davidenwang 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.

new monitors code looks good to me

@dcramer
dcramer merged commit df96944 into master Jan 5, 2023
@dcramer
dcramer deleted the feat/checkin-latest-api branch January 5, 2023 20:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Simplify monitor tracking API

2 participants