Skip to content

Conversation

dcramer
Copy link
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
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
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