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

chore(deps): Bump @metamask/smart-transactions-controller from 11.0.0 to ^13.0.0 #11140

Merged

Conversation

MajorLift
Copy link
Contributor

@MajorLift MajorLift commented Sep 10, 2024

Description

Bump @metamask/smart-transactions-controller from 11.0.0 to ^13.0.0

Related issues

Fixes: #11047

Manual testing steps

Screenshots/Recordings

Before

After

Pre-merge author checklist

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.

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

socket-security bot commented Sep 10, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@metamask/smart-transactions-controller@13.0.0 network 0 195 kB metamaskbot

🚮 Removed packages: npm/@metamask/smart-transactions-controller@11.0.0

View full report↗︎

@MajorLift MajorLift added the team-transactions Transactions team label Sep 10, 2024
@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch from 39bcf98 to 5f0eec9 Compare September 11, 2024 19:40
@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch 3 times, most recently from 058687b to b4bfd59 Compare September 12, 2024 06:48
@MajorLift MajorLift added DO-NOT-MERGE Pull requests that should not be merged needs-smoke-e2e labels Sep 12, 2024
@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch from b4bfd59 to 69970ad Compare September 12, 2024 07:02
@MajorLift MajorLift requested a review from a team September 12, 2024 08:02
@MajorLift MajorLift force-pushed the jongsun/chore/bump/smart-transactions-controller-13.0.0 branch from 8e48191 to 0bb5092 Compare September 12, 2024 08:03
@MajorLift MajorLift changed the base branch from jongsun/fix-composable-controller to jongsun/test/240911-e2e-fixes-for-10441 September 12, 2024 09:21
@MajorLift MajorLift force-pushed the jongsun/chore/bump/smart-transactions-controller-13.0.0 branch from 0bb5092 to e86ae9d Compare September 12, 2024 09:21
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from 4cfa4c0 to 393ff53 Compare September 12, 2024 09:55
@MajorLift MajorLift force-pushed the jongsun/chore/bump/smart-transactions-controller-13.0.0 branch from e86ae9d to 229eb5f Compare September 12, 2024 11:09
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from 393ff53 to 05ba0ff Compare September 12, 2024 15:11
@MajorLift MajorLift force-pushed the jongsun/chore/bump/smart-transactions-controller-13.0.0 branch from 65cd5f0 to a4cad2e Compare September 12, 2024 15:19
@MajorLift MajorLift removed the team-transactions Transactions team label Sep 13, 2024
@MajorLift MajorLift requested a review from a team September 13, 2024 05:02
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from 05ba0ff to 8436cc3 Compare September 13, 2024 05:05
@MajorLift MajorLift force-pushed the jongsun/chore/bump/smart-transactions-controller-13.0.0 branch from a4cad2e to b9370f3 Compare September 13, 2024 05:06
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from 9be6c47 to bb38098 Compare September 13, 2024 13:15
@MajorLift MajorLift force-pushed the jongsun/chore/bump/smart-transactions-controller-13.0.0 branch from b9370f3 to 6abc772 Compare September 13, 2024 13:20
…n mismatches

- These are because mobile's base-controller version is now ahead of the version used in the controllers
- These will be removed during the controller upgrades
Copy link
Contributor

https://bitrise.io/ Bitrise

🔄🔄🔄 pr_smoke_e2e_pipeline started on Bitrise...🔄🔄🔄

Commit hash: 6b2fd98
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/1475dbe5-db1f-42d6-9a68-5ceebcf12790

Note

  • This comment will auto-update when build completes
  • 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

@cryptodev-2s cryptodev-2s added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Sep 19, 2024
Copy link
Contributor

github-actions bot commented Sep 19, 2024

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 9be130a
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/e1b5c91d-170f-480e-befb-7c29ca9dd538

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

tommasini
tommasini previously approved these changes Sep 20, 2024
Copy link
Contributor

@tommasini tommasini left a comment

Choose a reason for hiding this comment

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

LGTM! Would like the eyes of @infiniteflower or someone from the transactions team!

@cryptodev-2s cryptodev-2s added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Sep 20, 2024
Copy link
Contributor

github-actions bot commented Sep 20, 2024

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: ef72a30
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/869f00da-3b43-4637-9819-a1cc341a8f61

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

tommasini
tommasini previously approved these changes Sep 20, 2024
@cryptodev-2s cryptodev-2s added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Sep 20, 2024
Copy link
Contributor

github-actions bot commented Sep 20, 2024

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: cc44b52
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/679a5929-645a-4930-8602-af0eb141ef6e

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

Copy link

@cryptodev-2s cryptodev-2s added this pull request to the merge queue Sep 20, 2024
Merged via the queue into main with commit 5275022 Sep 20, 2024
42 checks passed
@cryptodev-2s cryptodev-2s deleted the jongsun/chore/bump/smart-transactions-controller-13.0.0 branch September 20, 2024 21:23
@github-actions github-actions bot locked and limited conversation to collaborators Sep 20, 2024
@metamaskbot metamaskbot added the release-7.33.0 Issue or pull request that will be included in release 7.33.0 label Sep 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-7.33.0 Issue or pull request that will be included in release 7.33.0 Run Smoke E2E Triggers smoke e2e on Bitrise team-wallet-framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SmartTransactionsController upgrade from v11 to v13
5 participants