rollout_ui icon indicating copy to clipboard operation
rollout_ui copied to clipboard

Add and remove features from the UI

Open davejachimiak opened this issue 10 years ago • 5 comments

Adding features from the UI

In the case where RolloutUI is used in a system that eager loads enabled features for a user rather than asking whether a specific feature is enabled, developers can only create features through some sort of back-end mechanism like through the Rails console or a rake task.

This pull gives admins/developers the ability to add features directly from the Rollout UI, making it way easier to add features for such a system mentioned above.

Remove features from the UI

When a feature toggle is no longer necessary for a given feature, there is no way to clean up these features from the UI, let alone some back end mechanism.

This pull gives admins/developers the ability to remove features directly from the Rollout UI which will help keep our indexes clean.

davejachimiak avatar Jul 07 '14 20:07 davejachimiak

+1 remove features! Very needed

bomatson avatar Jul 11 '14 21:07 bomatson

+1 remove features

kbaum avatar Aug 27 '14 02:08 kbaum

+1 what do you think @jrallison can you merge this and give us 0.3.1 or 0.4?

mwynholds avatar Sep 08 '14 23:09 mwynholds

+1. One thing I noticed however, is that rollout also stores a comma-separated list of features in feature:__features__. But doesn't look like RolloutUI is using.

In case it's helpful, I wrote a tiny helper to remove features from Rollout

module RolloutHelper                                                                                                                                                                                               
  def self.remove_feature(feature)                                                                                                                                                                                 
    features = redis.get('feature:__features__').split(',')                                                                                                                                                 
    features.delete(feature)                                                                                                                                                                                       
    redis.set('feature:__features__', features.join(','))                                                                                                                                                   
    redis.srem('features', feature)                                                                                                                                                                         
  end                                                                                                                                                                                                              
end     

gingerlime avatar Jan 20 '15 10:01 gingerlime

Let's merge this, @jrallison

fredrik avatar Mar 18 '15 15:03 fredrik