-
Notifications
You must be signed in to change notification settings - Fork 86
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
fix(core): improve redirect behavior after creating account #1088
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
8 Skipped Deployments
|
🦋 Changeset detectedLatest commit: ec364c8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
859a2e7
to
f213960
Compare
efd4eff
to
8125b0d
Compare
8125b0d
to
a61f563
Compare
a61f563
to
8089ce1
Compare
e69cf33
to
6c7ee19
Compare
6c7ee19
to
6889b86
Compare
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.
👌
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.
looks good
This PR implements appearance of success banner by adding a search parameter to the URL. Thanks to this parameter we can control banner appearance. Another possible solution to implement was to use the context and change its status to Open to suggestions, thanks! |
core/app/[locale]/(default)/account/(tabs)/_components/message-wrapper.tsx
Outdated
Show resolved
Hide resolved
core/app/[locale]/(default)/account/(tabs)/_components/message-wrapper.tsx
Outdated
Show resolved
Hide resolved
9de4300
to
0382b31
Compare
0382b31
to
df59e5d
Compare
df59e5d
to
6479c93
Compare
6479c93
to
46cbee9
Compare
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.
Looks good. Just don't forget to fix linter errors)
46cbee9
to
30fbf5d
Compare
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.
Looks good 👍
be21cf6
to
94a90d3
Compare
94a90d3
to
ec364c8
Compare
⚡️🏠 Lighthouse reportLighthouse ran against https://catalyst-latest-oxswlvsl3-bigcommerce-platform.vercel.app 🖥️ DesktopWe ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:
📱 MobileWe ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:
|
What/Why?
This PR improves redirect behavior after creating account, namely, it adds a banner about the successful creation of an account.
Testing
locally
before:
create-account-redirect.mov
after:
fix-create-account-redirect.mov