-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial restructuring of the onboarding flow #8
Conversation
Codecov Report
@@ Coverage Diff @@
## main #8 +/- ##
==========================================
- Coverage 92.98% 91.32% -1.65%
==========================================
Files 26 24 -2
Lines 655 576 -79
==========================================
- Hits 609 526 -83
- Misses 46 50 +4
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Thank you for the initial changes, this already looks gerat @Supereg! 🚀 |
Initial restructuring of the onboarding flow
♻️ Current situation & Problem
As of right now, the app provides the onboarding flow as implemented in the template app.
💡 Proposed solution
This PR introduces the onboarding flow for NAMS by removing not needed features of the template and updating text accordingly.
⚙️ Release Notes
➕ Additional Information
Related PRs
--
Testing
TBA: test have to be adapted
Reviewer Nudging
Entry point to the review should be the
OnboardingFlow
view, where everything starts, even for the user.Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines: