Skip to content
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

Merged
merged 8 commits into from
May 21, 2024
Merged

Conversation

julesat22
Copy link
Contributor

@julesat22 julesat22 commented May 16, 2024

Description

Removed the "What's new" notification modal for the Portfolio. This is being removed because it was requested.

Open in GitHub Codespaces

Related issues

Fixes: 1150

Manual testing steps

  1. Download MM extension
  2. On a fresh install, scroll down to the "what's new" notification modal
  3. Check that the Portfolio notifications modal is no longer rendering

Screenshots/Recordings

Before

Screenshot 2024-05-16 at 9 06 39 AM

After

https://www.loom.com/share/451bc9aee9024097928a4ca7464e80e7

Pre-merge author checklist

  • I’ve followed MetaMask Coding Standards.
  • I've completed the PR template to the best of my ability
  • I’ve included tests if applicable
  • I’ve documented my code using JSDoc format if applicable
  • I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@julesat22 julesat22 requested a review from a team as a code owner May 16, 2024 17:24
@julesat22 julesat22 marked this pull request as draft May 16, 2024 17:24
Copy link
Contributor

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.

@julesat22 julesat22 marked this pull request as ready for review May 16, 2024 17:41
Copy link

codecov bot commented May 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.39%. Comparing base (c897b49) to head (4cc261c).

Current head 4cc261c differs from pull request most recent head 1b9ca46

Please upload reports for the commit 1b9ca46 to get more accurate results.

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.
📢 Have feedback on the report? Share it here.

@metamaskbot
Copy link
Collaborator

Builds ready [4cc261c]
Page Load Metrics (648 ± 486 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint6813186189
domContentLoaded104816115
load5526316481012486
domInteractive94816115
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0.00%)
  • ui: -200 Bytes (-0.00%)
  • common: -1.09 KiB (-0.02%)

@julesat22 julesat22 requested a review from ameliejyc May 17, 2024 18:37
@julesat22 julesat22 merged commit 0ef3c22 into develop May 21, 2024
70 of 72 checks passed
@julesat22 julesat22 deleted the MMPD-1150 branch May 21, 2024 15:31
@github-actions github-actions bot locked and limited conversation to collaborators May 21, 2024
@metamaskbot metamaskbot added the release-11.18.0 Issue or pull request that will be included in release 11.18.0 label May 21, 2024
@metamaskbot
Copy link
Collaborator

Builds ready [ef885ae]
Page Load Metrics (1130 ± 564 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint781711083014
domContentLoaded11281653
load65268411301175564
domInteractive11281653
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0.00%)
  • ui: -200 Bytes (-0.00%)
  • common: -1.09 KiB (-0.02%)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-11.18.0 Issue or pull request that will be included in release 11.18.0 team-portfolio
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants