-
Notifications
You must be signed in to change notification settings - Fork 457
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: Remove WC onboarding tooltip #2746
Conversation
Branch preview✅ Deploy successful! https://remove_wc_tooltip--walletweb.review-wallet-web.5afe.dev |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Coverage report
Test suite run success1046 tests passing in 143 suites. Report generated by 🧪jest coverage report action from 778af60 |
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. The tooltip is gone |
1815b73
to
0e118aa
Compare
Did something functional changed? or is just the e2e test? if it is just the test you can put it in QA done again |
5890749
to
18807e9
Compare
What it solves
As discussed on Slack we are removing the WC Onboarding tooltip.
How this PR fixes it
Removes the WC Onboarding Tooltip
Also fixes the Onboard Safe logo size on mobile
How to test it
Screenshots
Checklist