Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While integrating the pagination changes in #1020 I realised that we need the ability to find specific messages for display in the UI. Previously we just filtered the entire set, but after adding pagination to this endpoint, that is no longer possible.
This PR adds a
filter_by_ids
param toGET /v1/workspaces/:name/messages
, which accepts a list of IDs to filter by. I thought aboutGET /v1/workspaces/:name/messages/:id
— but I think given the semi-complex logic of parsing disparate prompts & outputs and matching them to messages, this might be simpler, as well as allowing other opportunities to find specific messages and highlight them in the dashboard at a later point.Examples
Without filtering:
Filtering by a single ID:
Filtering by multiple IDs:
Filtering by non-existent ID: