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

Enabling FTUE_AUTH onboarding variant #4872

Merged
merged 2 commits into from
Jan 6, 2022

Conversation

ouchadam
Copy link
Contributor

@ouchadam ouchadam commented Jan 6, 2022

Switches the build config to point to the FTUE_AUTH onboarding variant, this variant is a copy of the current (legacy login) with separate feature flags for future changes, eg #4727 and #4585

BEFORE SPLASH AFTER SPLASH
Screenshot_20220106_172749 Screenshot_20220106_173520
BEFORE FEATURES AFTER FEATURES
Screenshot_20220106_172757 Screenshot_20220106_173528

- this flow consistent of feature flag which can enable disable other parts, by default the I already have an account is enabled
@@ -0,0 +1 @@
Enabling new FTUE Auth onboarding base, includes the "I already have an account" button in the splash
Copy link
Contributor Author

Choose a reason for hiding this comment

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

wasn't sure if this was a .feature or .misc as we're only switching the build config value, could even be a new separate type (if we're able to have custom types)

Copy link
Member

Choose a reason for hiding this comment

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

OK for misc. I think we are limited by towncrier for possible extension but maybe not.

@ouchadam ouchadam requested a review from bmarty January 6, 2022 17:43
@github-actions
Copy link

github-actions bot commented Jan 6, 2022

Unit Test Results

  66 files  ±0    66 suites  ±0   49s ⏱️ -6s
135 tests ±0  135 ✔️ ±0  0 💤 ±0  0 ±0 
418 runs  ±0  418 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 0ea7fc2. ± Comparison against base commit f24c962.

@bmarty bmarty merged commit 3f9bea3 into develop Jan 6, 2022
@bmarty bmarty deleted the feature/adm/enabling-ftue-already-have-account branch January 6, 2022 19:57
@ouchadam ouchadam added the Z-FTUE Issue is relevant to the first time use project or experience label Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Z-FTUE Issue is relevant to the first time use project or experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants