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

Feature Flags! Cookies! #3156

Merged
merged 3 commits into from Sep 19, 2017
Merged

Feature Flags! Cookies! #3156

merged 3 commits into from Sep 19, 2017

Conversation

mgamini
Copy link
Contributor

@mgamini mgamini commented Sep 15, 2017

This moves feature flags and cookies to their own redux entries, and adds some sugar on top of the js-cookies lib.

@thesentinels
Copy link
Contributor

Thanks for the pull request! Here is what will happen next:

  1. Your PR will be reviewed by the maintainers
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!

@mgamini
Copy link
Contributor Author

mgamini commented Sep 15, 2017

Fixing something quickly

Garrett Amini added 2 commits September 19, 2017 09:27
Signed-off-by: Garrett Amini <garrett@chef.io>
Signed-off-by: Garrett Amini <garrett@chef.io>
Signed-off-by: Garrett Amini <garrett@chef.io>
@mgamini mgamini merged commit ac1134d into master Sep 19, 2017
@mgamini mgamini deleted the mgamini/feature-flags branch September 19, 2017 00:45
@christophermaier christophermaier added Type: Feature Issues that describe a new desired feature and removed C-feature labels Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Issues that describe a new desired feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants