Skip to content

Comments

PEDS-1490 - updated filter mode and guppy queries#668

Open
skyzhangty wants to merge 1 commit intopcdc_devfrom
feat/PEDS-1490
Open

PEDS-1490 - updated filter mode and guppy queries#668
skyzhangty wants to merge 1 commit intopcdc_devfrom
feat/PEDS-1490

Conversation

@skyzhangty
Copy link

@skyzhangty skyzhangty commented Oct 14, 2025

Screenshot 2025-10-14 at 7 32 34 AM Screenshot 2025-10-14 at 7 32 41 AM Screenshot 2025-10-14 at 7 32 59 AM Screenshot 2025-10-14 at 7 33 06 AM

@skyzhangty skyzhangty requested a review from grugna October 14, 2025 11:40
@grugna grugna requested a review from paulmurdoch19 October 14, 2025 14:44
@grugna
Copy link
Collaborator

grugna commented Oct 14, 2025

This looks good to me. @paulmurdoch19 could you take a second look at it? Specially the function used by the amanuensis jobs to fix and check the saved searches / filterset.

WARNING! Do not merge until uc-cdis/guppy#331 is merged.

@paulmurdoch19
Copy link

This looks good.

WARNING! when this is ready to be merged we need to create a migration in amanuensis to update all the filter-sets and update the validation job to use this new formating.

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