-
Notifications
You must be signed in to change notification settings - Fork 1
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
release: feature flag script #823
base: main
Are you sure you want to change the base?
Conversation
* feat: adjust update functionality * feat: create feature flag script 4460 * feat: remove deprecated flags * feat: remove soon to be deprecated flag * feat: add description * feat: include enablePartnerSettings 4460
✅ Deploy Preview for housing-sanjoseca-gov ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@mcgarrye I'm seeing 16 feature flags created rather than 15. Is that expected? |
yes, typo in the qa note |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🥅 but could you rerun tests to ensure the failure is flakiness
This PR addresses #(4460)
Description
This script will create the 16 feature flags in use across Core, HBA and Doorway in each database. Each has a simplified name, a readable description and is set to false. Following release and running of this script, the API must be called to associate the needed jurisdictions.
Related links:
Notion Documentation
Existing Flags Sheet
How Can This Be Tested/Reviewed?
yarn setup
/scriptRunner/addFeatureFlags
featureFlags/
to retrieve list of featureFlagsAuthor Checklist:
yarn generate:client
and/or created a migration when requiredReview Process: