-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update/add feature flag TM-1453 #626
Conversation
public/config/config.json
Outdated
"client_counts": false | ||
"client_counts": false, | ||
"cdo_season_filter": false, | ||
"cdo_bidding": false |
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.
Let's set both of these to true
for now (both here and config_dev.json
). We'll update them to false
in the State environment
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.
Done. And I messed up the merge conflicts with the snapshots I believe. What is the best way to handle that? Should I just accept all of the dev changes for snapshots? Or would it work itself out as long as there are no other merge conflicts and I run update snapshots?
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.
If you have merge conflicts with snapshots, you can just run tests again with --updateSnapshot
so that they get overwritten
Adding more flags for cdo bidding in addition to the feature flag for the cdo season filter on the CDO dashboard