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

feat(ui): Apply LaunchDarkly Feature Flags to ABP MMDL and CHIP Eligibility Pages and Cards #1569

Merged
merged 8 commits into from
Jan 3, 2025

Conversation

andieswift
Copy link
Collaborator

@andieswift andieswift commented Dec 30, 2024

Story: https://jiraent.cms.gov/browse/OY2-31722
Endpoint: https://d324rgo243013q.cloudfront.net/

Details

Wrapped the ABP MMDL and CHIP Eligibility cards in launch darkly flag: mmdlNotification so that when the flag is on, those options will not show

Implementation Notes

  • Used mmdlNotification flag rather than making my own flag
  • Added logic in Triage.js so that if the flag is on, any card with the description containing MMDL will not show
  • added logic to routes (matching the subdoc launch darkly code) so if the mmdlNotification flag is on, the landing pages redirecting the MMDL will not be added (aka those routes will redirect to 'Not Found')

Note: it seems my editor made some minor formating changes, I am sorry about that

Test Plan

  1. Leave mmdlNotification flag off
  2. Log into app and verify that on route: choices/spa/medicaid and choices/spa/chip still have the cards relating to MMDL
  3. turn the flag on
  4. Verify that cards referring to MMDL are no longer visible on above routes, and that those pages can no longer be accessed manually.

@andieswift andieswift self-assigned this Dec 30, 2024
Copy link

Endpoint URL - Environment not yet deployed, delete this comment and run the Pull Request workflow again after deploy is complete

@andieswift andieswift merged commit cb8171c into develop Jan 3, 2025
78 of 103 checks passed
@andieswift andieswift deleted the as-31722 branch January 3, 2025 14:57
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.

2 participants