Skip to content

Conversation

@quang-le
Copy link
Member

What problem is this PR solving? Explain here in one sentence.

Related JIRA tickets : POLIO-2039

Self proofreading checklist

  • Did I use eslint and ruff formatters?
  • Is my code clear enough and well documented?
  • Are my typescript files well typed?
  • New translations have been added or updated if new strings have been introduced in the frontend
  • My migrations file are included
  • Are there enough tests?
  • Documentation has been included (for new feature)

Doc

NA

Changes

  • Add switch to toggle between 2 types of LQAS filters
  • Add url parameters handle both types of filters
  • Remove backend filter on on_hold rounds for rounds api as it could create weird UI bugs
  • Fix link from afro map to use right params when redirecting

How to test

With a polio db with Lqas data:

  • Go to lqas (afro) map. Select a country and timeframe that match the DB data. Choose a country and click the button to LQAS in the pop up --> a new tab should open with the "classic" filters active for the campaign+ round
  • Go to lqas country map: try using both types of filters and toggling between them. Filter state is reset when toggling.
  • Repeat from private window with /dashboard/polio/embeddedLqasCountry/ and /dashboard/polio/embeddedLqasCountry/

Print screen / video

Screenshot 2025-12-31 at 14 51 58 Screenshot 2025-12-31 at 14 52 31
Screen.Recording.2025-12-31.at.15.04.31.mov

@quang-le quang-le added the release Should be released in production at next deploy label Dec 31, 2025
@quang-le quang-le requested a review from Bewi January 5, 2026 10:42
Copy link
Contributor

@Bewi Bewi left a comment

Choose a reason for hiding this comment

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

LGTM

@quang-le quang-le merged commit cfe11f3 into develop Jan 6, 2026
7 checks passed
@quang-le quang-le deleted the POLIO-2039_lqas_map_link branch January 6, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release Should be released in production at next deploy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants