Skip to content

Conversation

@JPrevost
Copy link
Member

What:

  • adds shared secret key access to flipflop controller in all
    environments (dev / test were previously always accessible)
  • travis defaults to running rake, new way is bin/rails test

Why:

  • the ability to easily toggle features for staff in PR and staging
    builds where they can confirm their current settings requires some
    form of dashboard and flipflop comes with one so why not use it

How:

  • setting ENV['FLIPFLOP_KEY'] and passing that value like
    /flipflop?flipflop_key=thatthingyouset will provide access to the
    dashboard where you can see all possible toggles and easily toggle
    features for your session
  • if you don't set ENV['FLIPFLOP_KEY'] all access is denied to the
    dashboard but you can still toggle features via the
    /toggle?feature=yourfeature route

What:

* adds shared secret key access to flipflop controller in all
  environments (dev / test were previously always accessible)
* travis defaults to running `rake`, new way is `bin/rails test`

Why:

* the ability to easily toggle features for staff in PR and staging
  builds where they can confirm their current settings requires some
  form of dashboard and flipflop comes with one so why not use it

How:

* setting `ENV['FLIPFLOP_KEY']` and passing that value like
  `/flipflop?flipflop_key=thatthingyouset` will provide access to the
  dashboard where you can see all possible toggles and easily toggle
  features for your session
* if you don't set `ENV['FLIPFLOP_KEY']` all access is denied to the
  dashboard but you can still toggle features via the
  `/toggle?feature=yourfeature` route
@gravesm gravesm temporarily deployed to mit-bento-staging-pr-168 March 31, 2017 18:19 Inactive
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 334f4a8 on flipflop_access_control into b143151 on master.

@JPrevost JPrevost requested a review from frrrances March 31, 2017 18:26
Copy link
Contributor

@frrrances frrrances left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you rock. 👍

@JPrevost JPrevost merged commit 66b61cd into master Mar 31, 2017
@JPrevost JPrevost deleted the flipflop_access_control branch March 31, 2017 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants