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

[NT-1257] Sign In With Apple Killswitch Feature Flag #1200

Merged
merged 3 commits into from
May 19, 2020

Conversation

justinswart
Copy link
Contributor

📲 What

Adds an ios_sign_in_with_apple feature flag that acts as a killswitch (hides the button if enabled).

🤔 Why

We will soon roll out Sign In With Apple in our next release and we would just like to be able to turn this off if necessary via a feature flag.

🛠 How

  • Added ios_sign_in_with_apple_killswitch feature flag on Optimizely.
  • Add behaviour to hide the Sign In With Apple button if this feature is enabled. If it is disabled or missing, the button will be shown.

👀 See

Killswitch enabled 🟢 Killswitch disabled 🔴 Killswitch missing 💭
image image image

✅ Acceptance criteria

  • Enable the ios_sign_in_with_apple_killswitch feature flag. Observe that upon re-launching the app, the Continue with Apple button is hidden.
  • Disable the ios_sign_in_with_apple_killswitch feature flag. Observe that upon re-launching the app, the Continue with Apple button is visible.
  • Temporary rename the feature flag so that it is seen as removed. Observe that upon re-launching the app, the Continue with Apple button is visible.

Copy link
Contributor

@ifbarrera ifbarrera left a comment

Choose a reason for hiding this comment

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

:shipit:

@justinswart justinswart merged commit 6929ad3 into master May 19, 2020
@justinswart justinswart deleted the NT-1257-sign-in-apple-feature-flag branch May 19, 2020 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants