Skip to content

Conversation

@segun
Copy link
Contributor

@segun segun commented Apr 15, 2024

Description

We should update content for:

when ppom returns warning as response

Open in GitHub Codespaces

Related issues

Fixes: #1758

Manual testing steps

  1. Add this custom network: https://base.drpc.org with chainId 8453
  2. Trigger a transaction from the test dapp -- see error
  3. For warning, you can edit the code to return a warning alert response and see the warning.

Screenshots/Recordings

Before

Screenshot 2024-04-15 at 22 37 08

After

Screenshot 2024-04-15 at 22 36 21

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.

@segun segun added team-confirmations-secure-ux-PR PRs from the confirmations team needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) labels Apr 15, 2024
@segun segun self-assigned this Apr 15, 2024
@segun segun requested review from a team as code owners April 15, 2024 21:39
@github-actions
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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should keep above comment, the issue I think is when i18n message keys should not be used like above. That makes it hard for i18n tool to discover un-used keys etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay sure. I wasn't sure why the comment was there.

jpuri
jpuri previously approved these changes Apr 16, 2024
Copy link
Contributor

@jpuri jpuri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except a small comment added.

blackdevelopa
blackdevelopa previously approved these changes Apr 16, 2024
@segun segun dismissed stale reviews from blackdevelopa and jpuri via c06b5d7 April 16, 2024 11:42
@segun segun force-pushed the olu/improve-content-banners-ppom branch from 1bb685c to c06b5d7 Compare April 16, 2024 11:42
@segun segun requested review from blackdevelopa and jpuri April 16, 2024 11:59
jpuri
jpuri previously approved these changes Apr 16, 2024
Copy link
Contributor

@jpuri jpuri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link

codecov bot commented Apr 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.58%. Comparing base (48a7969) to head (36e6a93).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #24037   +/-   ##
========================================
  Coverage    67.58%   67.58%           
========================================
  Files         1247     1247           
  Lines        48934    48934           
  Branches     12774    12774           
========================================
  Hits         33069    33069           
  Misses       15865    15865           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@segun segun requested a review from jpuri April 16, 2024 13:24
@metamaskbot
Copy link
Collaborator

Builds ready [32ff830]
Page Load Metrics (1515 ± 684 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint7860916012259
domContentLoaded118133178
load65367515151424684
domInteractive118033178
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0.00%)
  • ui: 22 Bytes (0.00%)
  • common: -1 Bytes (-0.00%)

@segun segun merged commit b373f16 into develop Apr 17, 2024
@segun segun deleted the olu/improve-content-banners-ppom branch April 17, 2024 11:07
@github-actions github-actions bot removed the needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) label Apr 17, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Apr 17, 2024
@metamaskbot metamaskbot added the release-11.16.0 Issue or pull request that will be included in release 11.16.0 label Apr 17, 2024
@metamaskbot
Copy link
Collaborator

Builds ready [36e6a93]
Page Load Metrics (2039 ± 618 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint823681697837
domContentLoaded147129147
load68333820391286618
domInteractive147129147
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0.00%)
  • ui: 22 Bytes (0.00%)
  • common: -1 Bytes (-0.00%)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-11.16.0 Issue or pull request that will be included in release 11.16.0 team-confirmations-secure-ux-PR PRs from the confirmations team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants