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-423] Remove Native Pledge Screen Feature Flag Override #906

Merged
merged 4 commits into from
Oct 25, 2019

Conversation

ifbarrera
Copy link
Contributor

@ifbarrera ifbarrera commented Oct 22, 2019

📲 What

Removes some hard-coded behavior that forces the nativeCheckoutPledgeView feature flag to default to false on release builds.

🤔 Why

This behavior was added to past builds to ensure that we could turn on the feature flag in the future without introducing incomplete behavior to older clients. Now that we are almost ready to release the next version of the app that will have a complete checkout experience, we can remove that hard-coded behavior. This will allow us to use the nativeCheckoutPledgeView feature flag to actually enable the native pledge screen feature for users on the next release, without affecting older builds.

🛠 How

Removed the hard-coded return false behavior for release builds. Also removed the associated tests.

👀 See

See how a Release build with the ios_native_checkout_pledge_view turned ON for my user respects the feature flag and takes the user to the new pledge screen:

T1BOJcguru

♿️ Accessibility

N/A

🏎 Performance

N/A

✅ Acceptance criteria

  • run a build in Release mode. Go to the production admin feature flags page, and turn the ios_native_checkout_pledge_view feature flag ON for just your user. Then, navigate to a project and choose a reward. You should be taken to the new pledge screen.

Copy link
Contributor

@Scollaco Scollaco left a comment

Choose a reason for hiding this comment

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

🎉 🌟 🎉 !

@ifbarrera ifbarrera merged commit 22466c1 into master Oct 25, 2019
@ifbarrera ifbarrera deleted the remove-hardcoded-flag branch October 25, 2019 16:15
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