-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[productcatalog] - generate error on feature flag #245
[productcatalog] - generate error on feature flag #245
Conversation
# Conflicts: # docker-compose.yml
After we merge #271 I'll update this branch from the base branch. |
# Conflicts: # .env
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
Can we have some user documentation for enabling this or will it be automatically set up soon after we merge this pr in a subsequent pr? |
That will come in a subsequent PR which also adds the actual feature flags to the database when the feature flag service starts. |
Changes
Generates an error on a specific product when a feature flag is enabled.
related issue: #162
Currently, this PR will not work since we have an issue that needs resolving with the feature flag service. Once that is resolved, we can finalize this PR.
For significant contributions please make sure you have completed the following items:
I will update the changelog after we resolve feature flag service communication issues
CHANGELOG.md
updated for non-trivial changes