Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Mar 1, 2024

This is the release candidate for version 7.18.0.

Changelog & Team Breakdown:
7.18.0-changelog.csv

@metamaskbot metamaskbot added the release-7.18.0 Issue or pull request that will be included in release 7.18.0 label Mar 2, 2024
@github-actions
Copy link
Contributor Author

github-actions bot commented Mar 2, 2024

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-commenter
Copy link

codecov-commenter commented Mar 2, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 43.75%. Comparing base (521ef63) to head (2594742).

Files Patch % Lines
app/core/Analytics/MetaMetrics.ts 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8821      +/-   ##
==========================================
- Coverage   43.75%   43.75%   -0.01%     
==========================================
  Files        1273     1273              
  Lines       31112    31113       +1     
  Branches     3165     3165              
==========================================
  Hits        13612    13612              
- Misses      16686    16688       +2     
+ Partials      814      813       -1     

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

This PR cherry-picks #8859

Co-authored-by: Nico MASSART <[email protected]>
Copy link
Contributor

@NicolasMassart NicolasMassart left a comment

Choose a reason for hiding this comment

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

Saving version number 1272 slot for Segment QA testing with prod creds

NicolasMassart and others added 3 commits March 7, 2024 15:53
Bump to V7.18.0 1272 - Segment QA testing with prod creds
Segment QA testing with prod creds
This PR cherry picks
#8775

Co-authored-by: MetaMask Bot <[email protected]>
Co-authored-by: metamaskbot <[email protected]>
Co-authored-by: sethkfman <[email protected]>
seaona and others added 2 commits March 7, 2024 12:34
## **Description**

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

## **Related issues**

Fixes:

## **Manual testing steps**

1. Go to this page...
2.
3.

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've clearly explained what problem this PR is solving and how it
is solved.
- [ ] I've linked related issues
- [ ] I've included manual testing steps
- [ ] I've included screenshots/recordings if applicable
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
- [ ] I’ve properly set the pull request status:
  - [ ] In case it's not yet "ready for review", I've set it to "draft".
- [ ] In case it's "ready for review", I've changed it from "draft" to
"non-draft".

## **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.

Co-authored-by: Jyoti Puri <[email protected]>
This PR cherry-picks #8861

Co-authored-by: Nico MASSART <[email protected]>
@A-Feder
Copy link

A-Feder commented Mar 8, 2024

QA notes, iOS 7.18 (1268):

❗️General View Scenario 1: Changing the device language from English to Russian.

COMMENT: ‘’Face ID’’ text overflowing on enable button, onboarding message on wallet view is displayed twice

❗Network Regression Scenario 1: I can remove a custom network added via the popular/custom network flow
COMMENT: "Do you want to remove this network" message doesn’t appear.

❗Network Regression Scenario 2: Adding a network via a Dapp
COMMENT: Couldn't connect to Aurora Mainnet, but it could be an issue with the dApp.

❗QR Scanner Scenario 2: I should be able to use the QR scanner on the send flow to scan an address.
COMMENT: The message 'recipient address is invalid' was displayed for some seconds before recognizing it.

❌ QR Scanner Scenario 5: When I scan an invalid address QR code using the in app camera I should see the appropriate warning message.
FAILED: the warning message does not appear

@ZbrancaI
Copy link

QA notes, Android 7.18 (build 1267):

❗️General View Scenario 1: Changing the device language from English to Russian.
COMMENT: The terms and conditions for the onboarding process are in English, not in Russian.

❗️Network Scenario 1.2: I can remove a custom network that was added via the custom network flow.
COMMENT: "Do you want to remove this network?" message doesn’t appear.

❗️Network Scenario 2: Adding a network via a Dapp.
COMMENT:

  • When connecting to Chainlist.wtf, the connect modal didn't appear;
  • I wasn't able to connect to Aurora Mainnet (it could be an issue with the dApp, as there was no problem connecting to Aurora Testnet);
  • The in-built browser sometimes remains on the main page, even if you insert a URL and press enter;
  • There was a UI issue, as the buttons were oriented too much on the bottom of the screen.

Screenshot_2024-03-07-18-22-00-256_io metamask (2)

❗️Browser Scenario 9: Navigate to opensea.io, rarible.com, pancakeswap.finance, app.uniswap.org, and connect wallet account.
COMMENT: When connecting to app.uniswap.org, the app closed, redirecting me to the phone's home screen. Switching to Ethereum Mainnet takes a long time.

❌ QR Scanner Scenario 5: When I scan an invalid address QR code using the in-app camera, I should see the appropriate warning message.
COMMENT: The warning message does not appear!

❗️QR Scanner Scenario 8: User scans MMM Receive QR code on different network.
COMMENT: for the Avalanche - Ethereum QR test, the message 'recipient address is invalid' was displayed for some seconds before recognizing it. (Maybe the network didn't switch yet, but it only happened for Ethereum)

GENERAL COMMENT:
❗️The app is unresponsive for a while when switching on the Ethereum Mainnet for some functionalities (QR scanner, the bottom menu).

This PR cherry-picks #8829 (removing all Mixpanel code)

Co-authored-by: Nico MASSART <[email protected]>
This PR cherry-picks #8877

Co-authored-by: salimtb <[email protected]>
@metamaskbot metamaskbot mentioned this pull request Mar 12, 2024
13 tasks
github-actions bot and others added 2 commits March 13, 2024 19:34
This PR cherry-picks #8888

Co-authored-by: Nico MASSART <[email protected]>
Co-authored-by: Cal Leung <[email protected]>
cc

## **Description**

- Bumped app build version from 1273 -> 1279, for RC 2

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

## **Related issues**

Fixes:

## **Manual testing steps**

1. Go to this page...
2.
3.

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've clearly explained what problem this PR is solving and how it
is solved.
- [ ] I've linked related issues
- [ ] I've included manual testing steps
- [ ] I've included screenshots/recordings if applicable
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
- [ ] I’ve properly set the pull request status:
  - [ ] In case it's not yet "ready for review", I've set it to "draft".
- [ ] In case it's "ready for review", I've changed it from "draft" to
"non-draft".

## **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.
This PR cherry-picks #8929

Co-authored-by: Xiaoming Wang <[email protected]>
Cal-L and others added 2 commits March 14, 2024 15:43
## **Description**

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

This PR does a few things:
- Remove redundant track event directory from Segment intergration
- Disable jest caching
- Fix audit issue for `follow-redirects`. Just regenerating yarn lock to
capture higher version

## **Related issues**

Fixes:

## **Manual testing steps**

1. Go to this page...
2.
3.

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've clearly explained what problem this PR is solving and how it
is solved.
- [ ] I've linked related issues
- [ ] I've included manual testing steps
- [ ] I've included screenshots/recordings if applicable
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
- [ ] I’ve properly set the pull request status:
  - [ ] In case it's not yet "ready for review", I've set it to "draft".
- [ ] In case it's "ready for review", I've changed it from "draft" to
"non-draft".

## **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.
)

**Description**
Fix to migration 29 with missing tokens related migrations, migration 30
with PPOM enabled by default, migration 31 to help user impacted by
missing migration on 29, migration 32 remove goerli

Also added migration 33, to recover from the crash of the users that the
migration 29 and 30 didn't had effect.

## **Related issues**

Fixes #8849

## **Manual testing steps**

1. Go to this page...
2.
3.

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've clearly explained what problem this PR is solving and how it
is solved.
- [ ] I've linked related issues
- [ ] I've included manual testing steps
- [ ] I've included screenshots/recordings if applicable
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
- [ ] I’ve properly set the pull request status:
  - [ ] In case it's not yet "ready for review", I've set it to "draft".
- [ ] In case it's "ready for review", I've changed it from "draft" to
"non-draft".

## **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.

---------

Co-authored-by: Cal-L <[email protected]>
## **Description**

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

Bump for v7.18.0 RC 3 1281

## **Related issues**

Fixes:

## **Manual testing steps**

1. Go to this page...
2.
3.

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've clearly explained what problem this PR is solving and how it
is solved.
- [ ] I've linked related issues
- [ ] I've included manual testing steps
- [ ] I've included screenshots/recordings if applicable
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
- [ ] I’ve properly set the pull request status:
  - [ ] In case it's not yet "ready for review", I've set it to "draft".
- [ ] In case it's "ready for review", I've changed it from "draft" to
"non-draft".

## **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.
Kylan Hurt and others added 2 commits March 18, 2024 13:30
Signed-off-by: Akintayo A. Olusegun <[email protected]>
Co-authored-by: Omri Dan <[email protected]>
Co-authored-by: Mark Stacey <[email protected]>
Co-authored-by: Matthew Walsh <[email protected]>
Co-authored-by: tommasini <[email protected]>
Co-authored-by: Nico MASSART <[email protected]>
Co-authored-by: Pedro Pablo Aste Kompen <[email protected]>
Co-authored-by: Jongsun Suh <[email protected]>
Co-authored-by: Xiaoming Wang <[email protected]>
Co-authored-by: Vivek <[email protected]>
Co-authored-by: sahar-fehri <[email protected]>
Co-authored-by: abretonc7s <[email protected]>
Co-authored-by: Jyoti Puri <[email protected]>
Co-authored-by: Cal Leung <[email protected]>
Co-authored-by: Curtis David <[email protected]>
Co-authored-by: sethkfman <[email protected]>
Co-authored-by: MetaMask Bot <[email protected]>
Co-authored-by: metamaskbot <[email protected]>
Co-authored-by: CW <[email protected]>
Co-authored-by: Sylva Elendu <[email protected]>
Co-authored-by: jiexi <[email protected]>
Co-authored-by: salimtb <[email protected]>
Co-authored-by: OGPoyraz <[email protected]>
Co-authored-by: Elliot Winkler <[email protected]>
Co-authored-by: Gustavo Antunes <[email protected]>
Co-authored-by: Timi Tejumola <[email protected]>
Co-authored-by: João Loureiro <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: sethkfman <[email protected]>
Co-authored-by: SamuelSalas <[email protected]>
Co-authored-by: seaona <[email protected]>
Co-authored-by: Victorien Gauch <[email protected]>
Co-authored-by: Olusegun Akintayo <[email protected]>
@sethkfman sethkfman marked this pull request as ready for review March 18, 2024 19:34
@sethkfman sethkfman requested a review from a team as a code owner March 18, 2024 19:34
@sethkfman sethkfman added the QA Passed QA testing has been completed and passed label Mar 18, 2024
sethkfman
sethkfman previously approved these changes Mar 18, 2024
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
Screenshot 2024-03-18 at 1 35 37 PM

@github-actions
Copy link
Contributor Author

github-actions bot commented Mar 18, 2024

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 9a9ecee
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/e743bdb5-c221-4bef-b369-5bcabde57873

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@sethkfman sethkfman added the team-mobile-platform Mobile Platform team label Mar 18, 2024
@github-actions github-actions bot requested a review from a team March 18, 2024 20:12
@github-actions
Copy link
Contributor Author

github-actions bot commented Mar 18, 2024

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 9f86026
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/d201372e-ed38-47ad-88cc-e4a71e2229af

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

## **Description**

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

Reverts PR #8919, which
broke E2E tests

## **Related issues**

Fixes:

## **Manual testing steps**

1. Go to this page...
2.
3.

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've clearly explained what problem this PR is solving and how it
is solved.
- [ ] I've linked related issues
- [ ] I've included manual testing steps
- [ ] I've included screenshots/recordings if applicable
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
- [ ] I’ve properly set the pull request status:
  - [ ] In case it's not yet "ready for review", I've set it to "draft".
- [ ] In case it's "ready for review", I've changed it from "draft" to
"non-draft".

## **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.
@github-actions
Copy link
Contributor Author

github-actions bot commented Mar 18, 2024

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 2594742
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/9b5eeb76-49f9-4647-a27a-858b9bfb1639

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@sonarqubecloud
Copy link

@Cal-L
Copy link
Contributor

Cal-L commented Mar 19, 2024

E2E tests - https://app.bitrise.io/build/8e50cc3a-63f5-419f-96cf-bb5f676b1d5d. iOS regression tests are failing but are coming from main and is not related to 7.18.0

Copy link
Contributor

@Cal-L Cal-L left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@chrisleewilcox chrisleewilcox left a comment

Choose a reason for hiding this comment

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

@Cal-L Cal-L merged commit 9739816 into main Mar 19, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Mar 19, 2024
@metamaskbot metamaskbot added the release-7.19.0 Issue or pull request that will be included in release 7.19.0 label Mar 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

QA Passed QA testing has been completed and passed release-7.18.0 Issue or pull request that will be included in release 7.18.0 release-7.19.0 Issue or pull request that will be included in release 7.19.0 team-mobile-platform Mobile Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.