Skip to content

feat: Support latest identifier on checkins#42819

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

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

Conversation

@dcramer
Copy link
Copy Markdown
Member

@dcramer dcramer commented Jan 5, 2023

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

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
Copy link
Copy Markdown
Contributor

@davidenwang davidenwang left a comment

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