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

7.16.0 #8464

Merged
merged 30 commits into from
Feb 28, 2024
Merged

7.16.0 #8464

merged 30 commits into from
Feb 28, 2024

Commits on Jan 30, 2024

  1. 7.16.0

    metamaskbot committed Jan 30, 2024
    Configuration menu
    Copy the full SHA
    0867e47 View commit details
    Browse the repository at this point in the history
  2. update changelog

    sethkfman committed Jan 30, 2024
    Configuration menu
    Copy the full SHA
    d8df54c View commit details
    Browse the repository at this point in the history
  3. chore: cherry-pick #8413 (#8475)

    This PR cherry-picks #8413
    
    Co-authored-by: LeoTM <1881059+leotm@users.noreply.github.com>
    github-actions[bot] and leotm authored Jan 30, 2024
    Configuration menu
    Copy the full SHA
    a6bae2d View commit details
    Browse the repository at this point in the history
  4. chore: cherry pick 8373 (#8477)

    ## **Description**
    
    cherry pick #8373 
    
    ## **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: LeoTM <1881059+leotm@users.noreply.github.com>
    sethkfman and leotm authored Jan 30, 2024
    Configuration menu
    Copy the full SHA
    ee451e4 View commit details
    Browse the repository at this point in the history
  5. bump version for RC 1 1248

    sethkfman committed Jan 30, 2024
    Configuration menu
    Copy the full SHA
    bc9cab7 View commit details
    Browse the repository at this point in the history

Commits on Jan 31, 2024

  1. chore: cherry-pick #8486 (#8487)

    This PR cherry-picks #8486
    
    Co-authored-by: Brian August Nguyen <brianacnguyen@gmail.com>
    github-actions[bot] and brianacnguyen authored Jan 31, 2024
    Configuration menu
    Copy the full SHA
    adc647e View commit details
    Browse the repository at this point in the history

Commits on Feb 1, 2024

  1. chore: cherry-pick #8157 (#8497)

    This PR cherry-picks #8157
    
    Co-authored-by: Matthew Walsh <matthew.walsh@consensys.net>
    github-actions[bot] and matthewwalsh0 authored Feb 1, 2024
    Configuration menu
    Copy the full SHA
    6996c3e View commit details
    Browse the repository at this point in the history
  2. chore: cherry-pick #8491 (#8498)

    This PR cherry-picks #8491
    
    Co-authored-by: Brian August Nguyen <brianacnguyen@gmail.com>
    github-actions[bot] and brianacnguyen authored Feb 1, 2024
    Configuration menu
    Copy the full SHA
    c1aebaa View commit details
    Browse the repository at this point in the history
  3. chore: cherry-pick #8469 (#8499)

    This PR cherry-picks #8469
    
    Co-authored-by: Omri Dan <61094771+omridan159@users.noreply.github.com>
    Co-authored-by: Nico MASSART <NicolasMassart@users.noreply.github.com>
    3 people authored Feb 1, 2024
    Configuration menu
    Copy the full SHA
    32dd416 View commit details
    Browse the repository at this point in the history
  4. chore: cherry-pick #8503 (#8506)

    This PR cherry-picks #8503
    
    Co-authored-by: Brian August Nguyen <brianacnguyen@gmail.com>
    github-actions[bot] and brianacnguyen authored Feb 1, 2024
    Configuration menu
    Copy the full SHA
    d59979e View commit details
    Browse the repository at this point in the history
  5. chore: cherry-pick #8504 (#8507)

    This PR cherry-picks #8504
    
    Co-authored-by: Brian August Nguyen <brianacnguyen@gmail.com>
    Co-authored-by: Cal Leung <cleun007@gmail.com>
    3 people authored Feb 1, 2024
    Configuration menu
    Copy the full SHA
    e71ee43 View commit details
    Browse the repository at this point in the history

Commits on Feb 2, 2024

  1. chore: cherry-pick #8509 - fix Sentry stack traces (#8511)

    ## **Description**
    
    This PR cherry picks #8509 
    
    <!--
    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.
    Cal-L authored Feb 2, 2024
    Configuration menu
    Copy the full SHA
    fb1f417 View commit details
    Browse the repository at this point in the history
  2. bumped up build to 1249 (#8505)

    Bumped up build version to 1249
    Andepande authored Feb 2, 2024
    Configuration menu
    Copy the full SHA
    a07606c View commit details
    Browse the repository at this point in the history

Commits on Feb 6, 2024

  1. chore: cherry-pick #8526 (#8530)

    This PR cherry-picks #8526
    
    Co-authored-by: Brian August Nguyen <brianacnguyen@gmail.com>
    github-actions[bot] and brianacnguyen authored Feb 6, 2024
    Configuration menu
    Copy the full SHA
    e03d2c7 View commit details
    Browse the repository at this point in the history
  2. chore: cherry-pick #8532 (#8535)

    This PR cherry-picks #8532
    
    Co-authored-by: Brian August Nguyen <brianacnguyen@gmail.com>
    github-actions[bot] and brianacnguyen authored Feb 6, 2024
    Configuration menu
    Copy the full SHA
    8c1d175 View commit details
    Browse the repository at this point in the history

Commits on Feb 13, 2024

  1. fix: Revert/8081 engine slice (#8576)

    ## **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 reverts #8081 .
    
    ## **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.
    Cal-L authored Feb 13, 2024
    Configuration menu
    Copy the full SHA
    13c3328 View commit details
    Browse the repository at this point in the history

Commits on Feb 15, 2024

  1. fix: Fix/1539 patch root file with controller data (#8578)

    ## **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 creates a migration that reconstructs the old root persisted
    structure. In the migration, it reconsolidates the controllers back into
    the root data. This solution is intended to be compatible when upgrading
    from any version to 7.16.0, especially for users who have already
    upgraded to 7.15.0.
    
    ## **Related issues**
    
    Fixes:
    
    ## **Manual testing steps**
    
    - Install 7.14.0 release and upgrade to 7.16.0
    - Install 7.15.0 release and upgrade to 7.16.0
    - Upgrade to 7.15.0 from 7.14.0 on Android and notice the vault
    corruption flow. Recover the account, then upgrade to 7.16.0
    
    The end behavior for all three scenarios is that the app should behave
    as expected and no data should be loss
    For testing support, please reach out to @MarioAslau or @Cal-L 
    
    ## **Screenshots/Recordings**
    
    <!-- If applicable, add screenshots and/or recordings to visualize the
    before and after of your change. -->
    
    ### **Before**
    
    <!-- [screenshots/recordings] -->
    
    ### **After**
    
    <!-- [screenshots/recordings] -->
    
    7.14.0 -> 7.16.0 is unaffected since the root architecture doesn't
    change and the migration just returns the original state
    
    iOS
    Going from 7.14 -> 7.15 -> 7.16. Same as going from 7.15 -> 7.16
    
    https://github.com/MetaMask/metamask-mobile/assets/10508597/1a80c32f-cc57-4a5f-9e5b-fb7052595434
    
    Going from 7.14 -> 7.16
    
    https://github.com/MetaMask/metamask-mobile/assets/10508597/1e5b9c8e-19c1-43f3-a05b-c4b2af47357d
    
    Android
    Same as iOS
    
    The video below shows that the app continues to function as expected
    when upgrading from 7.15.0 data -> 7.16.0 with the patch (Android)
    
    https://github.com/MetaMask/metamask-mobile/assets/10508597/a98256f6-b855-45a4-acfe-3bac891b9e16
    
    
    ## **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: tommasini <tommasini15@gmail.com>
    Co-authored-by: Aslau Mario-Daniel <marioaslau@gmail.com>
    3 people authored Feb 15, 2024
    Configuration menu
    Copy the full SHA
    f544ac2 View commit details
    Browse the repository at this point in the history
  2. Bump up build 7.16 build to 1263 (#8590)

    ## **Description**
    
    Bump up build 7.16 build to 1263
    Andepande authored Feb 15, 2024
    Configuration menu
    Copy the full SHA
    e4538fc View commit details
    Browse the repository at this point in the history

Commits on Feb 16, 2024

  1. chore: cherry-pick #8605 (#8609)

    This PR cherry-picks #8605
    
    Co-authored-by: João Tavares <joao.tavares@consensys.net>
    github-actions[bot] and cryptotavares authored Feb 16, 2024
    Configuration menu
    Copy the full SHA
    503140f View commit details
    Browse the repository at this point in the history
  2. version bump to 1265 (#8610)

    ## **Description**
    Version bump 1265
    <!--
    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: Nico MASSART <NicolasMassart@users.noreply.github.com>
    Andepande and NicolasMassart authored Feb 16, 2024
    Configuration menu
    Copy the full SHA
    5455a6e View commit details
    Browse the repository at this point in the history

Commits on Feb 22, 2024

  1. chore: merge main into 7.16.0 - resolve conflicts (#8679)

    ## **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?
    -->
    
    Fix merge conflicts with main for `release/7.16.0` branch
    
    ## **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.
    Cal-L authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    54e4f98 View commit details
    Browse the repository at this point in the history
  2. chore: chore/7.16.0 revert merge mainmerge (#8682)

    ## **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?
    -->
    
    Revert merge from main for 7.16.0
    
    ## **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.
    Cal-L authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    047fca5 View commit details
    Browse the repository at this point in the history
  3. chore: 7.16.0 merge conflicts from main (#8684)

    ## **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?
    -->
    
    Merge conflicts from main
    
    ## **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: LeoTM <1881059+leotm@users.noreply.github.com>
    Co-authored-by: sethkfman <10342624+sethkfman@users.noreply.github.com>
    Co-authored-by: sethkfman <seth.kaufman@consensys.net>
    Co-authored-by: Brian August Nguyen <brianacnguyen@gmail.com>
    Co-authored-by: sahar-fehri <sahar.fehri@consensys.net>
    Co-authored-by: metamaskbot <metamaskbot@users.noreply.github.com>
    Co-authored-by: Pedro Pablo Aste Kompen <wachunei@gmail.com>
    Co-authored-by: Matthew Walsh <matthew.walsh@consensys.net>
    Co-authored-by: Omri Dan <61094771+omridan159@users.noreply.github.com>
    Co-authored-by: Nico MASSART <NicolasMassart@users.noreply.github.com>
    Co-authored-by: Olusegun Akintayo <akintayo.segun@gmail.com>
    Co-authored-by: Brian Bergeron <brian.e.bergeron@gmail.com>
    Co-authored-by: tommasini <46944231+tommasini@users.noreply.github.com>
    Co-authored-by: Mike B <32695229+plasmacorral@users.noreply.github.com>
    Co-authored-by: Jyoti Puri <jyotipuri@gmail.com>
    Co-authored-by: George Marshall <george.marshall@consensys.net>
    Co-authored-by: Sylva Elendu <iamsylvaelendu@gmail.com>
    Co-authored-by: Nicholas Ellul <nicholas.ellul1@gmail.com>
    Co-authored-by: salimtb <salim.toubal@outlook.com>
    Co-authored-by: CW <chris.wilcox@consensys.net>
    Co-authored-by: Xiaoming Wang <dawnseeker8@gmail.com>
    Co-authored-by: Sébastien Van Eyck <sebastien.vaneyck@consensys.net>
    Co-authored-by: Sébastien Van Eyck <sebastien.vaneyck@gmail.com>
    Co-authored-by: OGPoyraz <omergoktugpoyraz@gmail.com>
    Co-authored-by: Mark Stacey <markjstacey@gmail.com>
    Co-authored-by: Vinicius Stevam <45455812+vinistevam@users.noreply.github.com>
    Co-authored-by: SamuelSalas <samuel.salas.reyes@gmail.com>
    Co-authored-by: Gustavo Antunes <17601467+gantunesr@users.noreply.github.com>
    Co-authored-by: João Tavares <joao.tavares@consensys.net>
    Co-authored-by: Frank von Hoven <141057783+frankvonhoven@users.noreply.github.com>
    Co-authored-by: João Loureiro <1649425+jpcloureiro@users.noreply.github.com>
    Co-authored-by: MetaMask Bot <37885440+metamaskbot@users.noreply.github.com>
    Co-authored-by: witmicko <witmicko@users.noreply.github.com>
    Co-authored-by: Derek Brans <dbrans@gmail.com>
    Co-authored-by: Curtis David <Curtis.David7@gmail.com>
    Co-authored-by: Owen Craston <owen.craston@consensys.net>
    Configuration menu
    Copy the full SHA
    7948614 View commit details
    Browse the repository at this point in the history
  4. chore: Fix more merge conflicts with main (#8686)

    ## **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?
    -->
    
    Fix more merge conflicts with main on 7.16.0
    
    ## **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: LeoTM <1881059+leotm@users.noreply.github.com>
    Co-authored-by: sethkfman <10342624+sethkfman@users.noreply.github.com>
    Co-authored-by: sethkfman <seth.kaufman@consensys.net>
    Co-authored-by: Brian August Nguyen <brianacnguyen@gmail.com>
    Co-authored-by: sahar-fehri <sahar.fehri@consensys.net>
    Co-authored-by: metamaskbot <metamaskbot@users.noreply.github.com>
    Co-authored-by: Pedro Pablo Aste Kompen <wachunei@gmail.com>
    Co-authored-by: Matthew Walsh <matthew.walsh@consensys.net>
    Co-authored-by: Omri Dan <61094771+omridan159@users.noreply.github.com>
    Co-authored-by: Nico MASSART <NicolasMassart@users.noreply.github.com>
    Co-authored-by: Olusegun Akintayo <akintayo.segun@gmail.com>
    Co-authored-by: Brian Bergeron <brian.e.bergeron@gmail.com>
    Co-authored-by: tommasini <46944231+tommasini@users.noreply.github.com>
    Co-authored-by: Mike B <32695229+plasmacorral@users.noreply.github.com>
    Co-authored-by: Jyoti Puri <jyotipuri@gmail.com>
    Co-authored-by: George Marshall <george.marshall@consensys.net>
    Co-authored-by: Sylva Elendu <iamsylvaelendu@gmail.com>
    Co-authored-by: Nicholas Ellul <nicholas.ellul1@gmail.com>
    Co-authored-by: salimtb <salim.toubal@outlook.com>
    Co-authored-by: CW <chris.wilcox@consensys.net>
    Co-authored-by: Xiaoming Wang <dawnseeker8@gmail.com>
    Co-authored-by: Sébastien Van Eyck <sebastien.vaneyck@consensys.net>
    Co-authored-by: Sébastien Van Eyck <sebastien.vaneyck@gmail.com>
    Co-authored-by: OGPoyraz <omergoktugpoyraz@gmail.com>
    Co-authored-by: Mark Stacey <markjstacey@gmail.com>
    Co-authored-by: Vinicius Stevam <45455812+vinistevam@users.noreply.github.com>
    Co-authored-by: SamuelSalas <samuel.salas.reyes@gmail.com>
    Co-authored-by: Gustavo Antunes <17601467+gantunesr@users.noreply.github.com>
    Co-authored-by: João Tavares <joao.tavares@consensys.net>
    Co-authored-by: Frank von Hoven <141057783+frankvonhoven@users.noreply.github.com>
    Co-authored-by: João Loureiro <1649425+jpcloureiro@users.noreply.github.com>
    Co-authored-by: MetaMask Bot <37885440+metamaskbot@users.noreply.github.com>
    Co-authored-by: witmicko <witmicko@users.noreply.github.com>
    Co-authored-by: Derek Brans <dbrans@gmail.com>
    Co-authored-by: Curtis David <Curtis.David7@gmail.com>
    Co-authored-by: Owen Craston <owen.craston@consensys.net>
    Configuration menu
    Copy the full SHA
    ec51f71 View commit details
    Browse the repository at this point in the history
  5. chore: chore/7.16.0 revert more prs (#8687)

    ## **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?
    -->
    
    Revert conflict comments on 7.16.0
    
    ## **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.
    Cal-L authored Feb 22, 2024
    Configuration menu
    Copy the full SHA
    74e9eb9 View commit details
    Browse the repository at this point in the history

Commits on Feb 26, 2024

  1. merge 'main' into release

    sethkfman committed Feb 26, 2024
    Configuration menu
    Copy the full SHA
    2cd8783 View commit details
    Browse the repository at this point in the history

Commits on Feb 27, 2024

  1. Configuration menu
    Copy the full SHA
    73771cb View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    1225bfd View commit details
    Browse the repository at this point in the history

Commits on Feb 28, 2024

  1. Configuration menu
    Copy the full SHA
    b3f356b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e7c3c4b View commit details
    Browse the repository at this point in the history