-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
feat: new onboarding #9066
feat: new onboarding #9066
Conversation
* feat: Adds onboarding views after signup * Updates i18n strings * Review fixes * feat: Adds assets used in the onboarding pages (#8935) * feats: Adds assets used in the onboarding pages * Adds icons and translations * Update app/javascript/dashboard/i18n/locale/en/signup.json * Updates locale string * Review fixes * Fixes layout on large and extra large screens * Reverts unwanted changes * Fixes styles on signup page * Adds signup page APIs * Update app/javascript/dashboard/routes/index.js * Updates signup page
* feat: Adds steps to show onboarding view after signup * Updates routes * Review fixes * chore: Adds helpers needed for onboarding views (#8937) * chore: Adds helpers needed for onboarding views * Update OnboardingHelper.js * Update OnboardingHelper.spec.js * Update BrowserHelper.spec.js * Update app/javascript/v3/helpers/BrowserHelper.js Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com> * Update app/javascript/v3/helpers/OnboardingHelper.js Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com> * Updates helper * Update app/javascript/dashboard/constants/globals.js * Update app/javascript/v3/helpers/BrowserHelper.js * Update app/javascript/v3/helpers/specs/BrowserHelper.spec.js * Adds case for handling empty values * Lint error fixes --------- Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com> --------- Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com>
…t into feat/onboarding-cw-2667
This reverts commit 898376c.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@scmmishra Please take a look at the comments.
|
🐢 Turtley slow progress alert! This pull request has been idle for over 30 days. Can we please speed things up and either merge it or release it back into the wild? |
Closing for now, will re-open if required |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
CleanShot.2024-03-08.at.16.08.19.mp4
Fixes https://linear.app/chatwoot/issue/CW-2887/onboarding-ui