Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(breaking) O3-2899 - queue service should not limit to only active qu… #57

Merged
merged 4 commits into from
Feb 26, 2024

Conversation

mseaton
Copy link
Member

@mseaton mseaton commented Feb 26, 2024

…eue entries by default

This is a breaking change, as existing queries to the service, including usage by the REST endpoint, will not include both active and ended queue entries by default, unless the "isActive=true" parameter is passed in. We will need to update the frontend code to ensure this is being passed in, but given this frontend code is a work in progress, and the existing code was causing a lot of confusion and errors, I wanted to get this fixed.

Copy link
Member

@cioan cioan left a comment

Choose a reason for hiding this comment

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

This makes sense to me, @mseaton ! Thanks!

@cioan
Copy link
Member

cioan commented Feb 26, 2024

@mseaton mseaton merged commit f58dd8d into main Feb 26, 2024
1 check passed
@mseaton mseaton deleted the O3-2899-1 branch February 26, 2024 22:40
@mseaton mseaton restored the O3-2899-1 branch February 26, 2024 22:40
@mseaton mseaton deleted the O3-2899-1 branch February 26, 2024 22:40
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.

3 participants