jrallison / rollout_ui

RolloutUI: A slick way to rollout features in your web app.
MIT License
137 stars 86 forks source link

MonkeyPatch: don't write feature if it already exists. #28

Closed jelmersnoeck closed 8 years ago

jelmersnoeck commented 10 years ago

Always writing is causing errors when you're using Rollout in a case where you want certain parts to only read from slaves and other parts to only write to your master.

As we're overwriting the standard Rollout.active? class, checking for the active state in a read-only slave would cause a Redis error. Only writing this feature when it doesn't exist resolves this issue.