-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
refactor: removed the whats new modal for portfolio #24566
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #24566 +/- ##
===========================================
+ Coverage 67.37% 67.39% +0.02%
===========================================
Files 1289 1289
Lines 50221 50212 -9
Branches 13005 13008 +3
===========================================
+ Hits 33836 33840 +4
+ Misses 16385 16372 -13 ☔ View full report in Codecov by Sentry. |
Builds ready [4cc261c]
Page Load Metrics (648 ± 486 ms)
Bundle size diffs [🚀 Bundle size reduced!]
|
Builds ready [ef885ae]
Page Load Metrics (1130 ± 564 ms)
Bundle size diffs [🚀 Bundle size reduced!]
|
Description
Removed the "What's new" notification modal for the Portfolio. This is being removed because it was requested.
Related issues
Fixes: 1150
Manual testing steps
Screenshots/Recordings
Before
After
https://www.loom.com/share/451bc9aee9024097928a4ca7464e80e7
Pre-merge author checklist
Pre-merge reviewer checklist