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

VAMC real-time banners tech debt cleanup: remove flipper / feature toggle logic and flippers/ toggles #19468

Closed
5 tasks
Tracked by #19466
jilladams opened this issue Oct 9, 2024 · 6 comments
Labels
Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Ruby sitewide VA.gov frontend CMS team practice area VAMC real-time banners VAMC banners (without situation updates so far) that publish via vets-website rather than content bu VAMC CMS managed product owned by Facilities team

Comments

@jilladams
Copy link
Contributor

jilladams commented Oct 9, 2024

If we have created flippers for banner_use_alternative_banners in vets-api or CMS feature toggles, we need to:

Acceptance criteria

@jilladams jilladams changed the title Alternative banners tech debt cleanup: remove flipper logic and flippers Alternative banners tech debt cleanup: remove flipper / feature toggle logic and flippers/ toggles Oct 9, 2024
@jilladams jilladams added VAMC CMS managed product owned by Facilities team Drupal engineering CMS team practice area Needs refining Issue status Ruby VAMC real-time banners VAMC banners (without situation updates so far) that publish via vets-website rather than content bu Facilities Facilities products (VAMC, Vet Center, etc) labels Oct 9, 2024
@jilladams jilladams removed the Needs refining Issue status label Nov 6, 2024
@Agile6MSkinner
Copy link

Calling this refined, but @SnowboardTechie needs to get the Flipper/Toggle names into the ticket. If they're not here yet, go after him.

@SnowboardTechie
Copy link

@jilladams
Copy link
Contributor Author

WHen we pick this up, double check with Bryan / Daniel that the listed flipper names haven't changed - should be easy to tell in prod:

@jilladams jilladams changed the title Alternative banners tech debt cleanup: remove flipper / feature toggle logic and flippers/ toggles VAMC real-time banners tech debt cleanup: remove flipper / feature toggle logic and flippers/ toggles Nov 22, 2024
@jilladams jilladams added the VA.gov frontend CMS team practice area label Dec 18, 2024
@jilladams
Copy link
Contributor Author

We decided today that based on Architecture intent for real-time banners, we need to keep both the CMS feature toggle and the vets-api flipper long-term, so we can cut back to content-build version of banners in the future as needed. I've rolled a documentation update about that into #11591 for the flipper.

This ticket will close as no-op.

However, one question first: @SnowboardTechie does this AC need to do anything different in light of the rest of this comment? "Remove context from Banners if no longer being utilized now that path is parsed department-of-veterans-affairs/vets-api#19897"

@SnowboardTechie
Copy link

@jilladams I believe that AC could be removed from this issue. It fits better with #20093 and can be included with that work.

@jilladams
Copy link
Contributor Author

🙌

@jilladams jilladams closed this as not planned Won't fix, can't repro, duplicate, stale Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Ruby sitewide VA.gov frontend CMS team practice area VAMC real-time banners VAMC banners (without situation updates so far) that publish via vets-website rather than content bu VAMC CMS managed product owned by Facilities team
Projects
None yet
Development

No branches or pull requests

3 participants