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

deps: @metamask/transaction-controller@^29.1.0->^30.0.0 #342

Merged
merged 2 commits into from
Jun 3, 2024

Conversation

legobeat
Copy link
Contributor

@legobeat legobeat commented May 29, 2024

Ensuring latest version is of @metamask/transaction-controller can be used with this package.


Blocking


@legobeat legobeat changed the title deps: @metamak/transaction-controller@^29.1.0->^30.0.0 deps: @metamask/transaction-controller@^29.1.0->^30.0.0 May 29, 2024
@legobeat legobeat force-pushed the deps-transactions-controller-30 branch from 3112700 to 1bcb155 Compare May 29, 2024 11:57
Copy link

socket-security bot commented May 29, 2024

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

Package New capabilities Transitives Size Publisher
npm/@metamask/[email protected] None +1 50 kB

🚮 Removed packages: npm/[email protected], npm/[email protected]

View full report↗︎

@legobeat legobeat force-pushed the deps-transactions-controller-30 branch from 16429d7 to 3ec18be Compare May 30, 2024 07:44
@legobeat legobeat mentioned this pull request May 30, 2024
1 task
@legobeat legobeat requested review from a team, dan437 and dbrans and removed request for dan437 and dbrans May 31, 2024 10:33
@legobeat
Copy link
Contributor Author

@dan437

@legobeat legobeat merged commit f37c409 into MetaMask:main Jun 3, 2024
16 checks passed
dan437 pushed a commit to MetaMask/metamask-extension that referenced this pull request Jun 5, 2024
## **Description**

This effectively relates to two separate fixes for issues with polling
behavior.

-
https://github.com/MetaMask/smart-transactions-controller/releases/tag/v10.1.1

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/25030?quickstart=1)


## **Related issues**
- Follow-up to and aligns with #24861
- #24913

### Contents Highlights
-
MetaMask/smart-transactions-controller@v10.0.1...v10.1.1
- MetaMask/smart-transactions-controller#347
- MetaMask/smart-transactions-controller#339
- MetaMask/smart-transactions-controller#342

## **Manual testing steps**

## **Screenshots/Recordings**

### **Before**

### **After**

## **Pre-merge author checklist**

- [x] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)).
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.
danjm pushed a commit to MetaMask/metamask-extension that referenced this pull request Jun 13, 2024
This effectively relates to two separate fixes for issues with polling
behavior.

-
https://github.com/MetaMask/smart-transactions-controller/releases/tag/v10.1.1

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/25030?quickstart=1)

- Follow-up to and aligns with #24861
- #24913

-
MetaMask/smart-transactions-controller@v10.0.1...v10.1.1
- MetaMask/smart-transactions-controller#347
- MetaMask/smart-transactions-controller#339
- MetaMask/smart-transactions-controller#342

- [x] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

- [ ] 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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants