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

Allow for dynamic feature flags #6808

Merged
merged 4 commits into from Feb 27, 2019

Conversation

mistercrunch
Copy link
Member

Giving more control over feature flags, allowing administrator to define
custom logic around whether features are enabled for particular users /
roles.

The exposed function can be used for things like:

  • progressive rollout of features (1%, 5%, 50%, 100%)
  • experimentation
  • role-based feature affectation (only admins see a particular feature)

Giving more control over feature flags, allowing administrator to define
custom logic around whether features are enabled for particular users /
roles.

The exposed function can be used for things like:
* progressive rollout of features (1%, 5%, 50%, 100%)
* experimentation
* role-based feature affectation (only admins see a particular feature)
@codecov-io
Copy link

codecov-io commented Feb 2, 2019

Codecov Report

Merging #6808 into master will increase coverage by 0.28%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6808      +/-   ##
==========================================
+ Coverage   56.03%   56.32%   +0.28%     
==========================================
  Files         527      527              
  Lines       23286    23536     +250     
  Branches     2788     2782       -6     
==========================================
+ Hits        13049    13257     +208     
- Misses       9827     9867      +40     
- Partials      410      412       +2
Impacted Files Coverage Δ
superset/config.py 93.71% <100%> (+0.08%) ⬆️
superset/__init__.py 74.43% <100%> (+0.83%) ⬆️
superset/views/base.py 69.1% <100%> (ø) ⬆️
superset/models/sql_lab.py 94.66% <0%> (-0.99%) ⬇️
superset/assets/src/explore/controls.jsx 42.4% <0%> (-0.35%) ⬇️
superset/db_engine_specs.py 53.82% <0%> (-0.29%) ⬇️
superset/assets/src/visualizations/nvd3/utils.js 11.57% <0%> (-0.1%) ⬇️
superset/sql_parse.py 99.15% <0%> (-0.05%) ⬇️
superset/views/__init__.py 100% <0%> (ø) ⬆️
...explore/components/controls/ColorSchemeControl.jsx 46.66% <0%> (ø) ⬆️
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f3d2bc...30daf3e. Read the comment docs.

superset/config.py Outdated Show resolved Hide resolved
superset/config.py Outdated Show resolved Hide resolved
superset/config.py Outdated Show resolved Hide resolved
superset/__init__.py Outdated Show resolved Hide resolved
@kristw kristw added the enhancement:request Enhancement request submitted by anyone from the community label Feb 7, 2019
@mistercrunch
Copy link
Member Author

Addressed all comments

@mistercrunch mistercrunch merged commit 3ae02d1 into apache:master Feb 27, 2019
@mistercrunch mistercrunch deleted the dynamic_feature_flags branch February 27, 2019 23:11
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.34.0 labels Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels enhancement:request Enhancement request submitted by anyone from the community 🚢 0.34.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants