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.

Sign up for GitHub

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

Jump to bottom

Fix jsonb filtering in scheduler #3627

Closed
jpbruinsslot opened this issue Oct 7, 2024 · 3 comments · Fixed by #3643
Closed

Fix jsonb filtering in scheduler #3627

jpbruinsslot opened this issue Oct 7, 2024 · 3 comments · Fixed by #3643
Assignees
@jpbruinsslot
Labels
mula Issues related to the scheduler

Comments

@jpbruinsslot
Copy link
Contributor

jpbruinsslot commented Oct 7, 2024 •
edited
Loading

Originally posted by @Souf149 in #3299 (comment)
@jpbruinsslot jpbruinsslot self-assigned this Oct 7, 2024
@jpbruinsslot jpbruinsslot added the mula Issues related to the scheduler label Oct 7, 2024
@jpbruinsslot jpbruinsslot added this to KAT Oct 7, 2024
@github-project-automation github-project-automation bot moved this to Incoming features / Need assessment in KAT Oct 7, 2024
@jpbruinsslot jpbruinsslot moved this from Incoming features / Need assessment to In Progress in KAT Oct 7, 2024
@jpbruinsslot jpbruinsslot mentioned this issue Oct 8, 2024
Merged
9 tasks
@jpbruinsslot
Copy link
Contributor Author

I'm seeing some weird string interpretation when using this from the API side. Internally tests are working and queries are constructed correctly.

@jpbruinsslot
Copy link
Contributor Author

I'm seeing some weird string interpretation when using this from the API side. Internally tests are working and queries are constructed correctly.

Had to do with curl escaping " and how fastapi inteprets it. Usage within python should just work, and wrote tests for it.

@jpbruinsslot
Copy link
Contributor Author

jpbruinsslot commented Oct 9, 2024 •
edited
Loading

Having issues with the CI test runner at the moment
Fixed

@jpbruinsslot jpbruinsslot moved this from In Progress to Review in KAT Oct 9, 2024
@ammar92 ammar92 moved this from Review to QA review / functional testing in KAT Oct 10, 2024
@stephanie0x00 stephanie0x00 moved this from QA review / functional testing to Ready for merge in KAT Oct 14, 2024
@github-project-automation github-project-automation bot moved this from Ready for merge to Done in KAT Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jpbruinsslot jpbruinsslot

Labels
mula Issues related to the scheduler
Projects
KAT
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix jsonb 'contained by' query minvws/nl-kat-coordination
1 participant
@jpbruinsslot