department-of-veterans-affairs / va.gov-cms

Editor-centered management for Veteran-centered content.
https://prod.cms.va.gov
GNU General Public License v2.0
98 stars 68 forks source link

FE: Establish criteria to remove flipper facility_locator_ppms_forced_unique_id #11593

Closed dsinla closed 1 year ago

dsinla commented 1 year ago

Description

As we investigate each Facilities related Flipper, document the criteria and timeline for when it should be removed.

Refer to https://github.com/department-of-veterans-affairs/va.gov-cms/issues/10656

Acceptance Criteria

CMS Team

Please check the team(s) that will do this work.

maxx1128 commented 1 year ago

I checked content-build and vets-website and found no reference to this flipper in any code files. It's already been removed from the front end, so the only step left is to remove the toggle from vets-api which is an easy change.

jilladams commented 1 year ago

@maxx1128 just saw your Slack DMs. One note here -- we found previously that the name of the flipper as snake case won't always appear in vets-website code. Instead, notes here on how it gets translated to camelCase. That camelCase reference is really what we need to confirm doesn't exist anywhere in the front end.

Can you take another look with that linked comment in mind and confirm if this flipper is safe to remove. (cc @ryguyk )

jilladams commented 1 year ago

Also noting, this flipper is currently enabled in production: https://api.va.gov/flipper/features

maxx1128 commented 1 year ago

@jilladams I checked for snake case and camel case on the front-end and neither one is present in the front-end.

maxx1128 commented 1 year ago

The branches for removing the flipper from the needed repos are below. No pull requests have been made with them yet, but can be once this flipper is approved for removal.

  1. vets-api - va-11593-remove-ppms-forced-unique-id-feature
jilladams commented 1 year ago

@maxx1128 that in mind: can this ticket be closed?

maxx1128 commented 1 year ago

@jilladams Yes I think it can be closed as is since it's having zero effect on the front-end from what I can see.

kmariepat-cityfriends commented 1 year ago

Closing this card, Implementation will be completed on #13239