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

feat: add linea mainnet alert message #6632

Merged
merged 16 commits into from
Jun 20, 2023

Conversation

VGau
Copy link
Contributor

@VGau VGau commented Jun 19, 2023

Development & PR Process

  1. Follow MetaMask Mobile Coding Standards
  2. Add release-xx label to identify the PR slated for a upcoming release (will be used in release discussion)
  3. Add needs-dev-review label when work is completed
  4. Add needs-qa label when dev review is completed
  5. Add QA Passed label when QA has signed off

Description

  • Added a warning message when we switch to Linea mainnet and this network is not ready.

⚠️ This branch is based on feat/551-add-linea-mainnet branch because we need to have the linea mainnet network implemented to add the warning message.

Screenshots/Recordings

0xaQaQud2k.mp4

If applicable, add screenshots and/or recordings to visualize the before and after of your change

Issue

Progresses #???

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

@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.

@VGau VGau marked this pull request as ready for review June 19, 2023 09:33
@VGau VGau requested a review from a team as a code owner June 19, 2023 09:33
@sethkfman sethkfman added release-7.2.0 needs-qa Any New Features that needs a full manual QA prior to being added to a release. Spot Check on the Release Build If a ticket doesn't require feature QA, but does require some form of manual spot checking labels Jun 20, 2023
Copy link
Contributor

@sethkfman sethkfman left a comment

Choose a reason for hiding this comment

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

LGTM

@sethkfman
Copy link
Contributor

@cortisiko Hey, I want to review this PR with you before we merge for 7.2.0.

@cortisiko cortisiko removed the needs-qa Any New Features that needs a full manual QA prior to being added to a release. label Jun 20, 2023
@cortisiko
Copy link
Member

@sethkfman @VGau one small feedback here: we would need to ensure that the newly added strings are translated. Not a blocker for getting this PR merged into main, but we should aim to have the translations as part of the release. I will approve

@sethkfman sethkfman merged commit 6a5ca48 into MetaMask:main Jun 20, 2023
9 of 10 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-7.2.0 Spot Check on the Release Build If a ticket doesn't require feature QA, but does require some form of manual spot checking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants