Commit 69d1ae2
authored
fix: cp-7.60.0 predict withdraw using gas station (MetaMask#23255)
## **Description**
Fix Predict withdraw when using gas station with insufficient existing
token balance.
Bump `transaction-controller` and `transaction-pay-controller`.
## **Changelog**
CHANGELOG entry: null
## **Related issues**
Fixes: MetaMask#23137 MetaMask#23126
## **Manual testing steps**
## **Screenshots/Recordings**
### **Before**
### **After**
## **Pre-merge author checklist**
- [x] I’ve followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.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-mobile/blob/main/.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.
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> Prevents gas-fee token injection/external sign when
`txMeta.isGasFeeTokenIgnoredIfBalance` is set; wires Remote Feature Flag
messenger action; bumps transaction/transaction-pay controllers.
>
> - **Confirmations**:
> - Update `useTransactionConfirm` to skip adding `batchTransactions`
and `isExternalSign` when `txMeta.isGasFeeTokenIgnoredIfBalance` is
true.
> - Include `isGasFeeTokenIgnoredIfBalance` in metadata handling for
smart transactions and 7702 flows.
> - **Tests**:
> - Add tests ensuring no gas-fee token batching or external signing
when `isGasFeeTokenIgnoredIfBalance` is set.
> - **Messaging**:
> - Allow `RemoteFeatureFlagController:getState` in
`transaction-controller` init messenger.
> - **Dependencies**:
> - Bump `@metamask/transaction-controller` to `62.3.0` and
`@metamask/transaction-pay-controller` to `10.1.0` (with corresponding
lockfile updates).
>
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
ccbc4d8. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->1 parent d910f89 commit 69d1ae2
5 files changed
Lines changed: 200 additions & 28 deletions
File tree
- app
- components/Views/confirmations/hooks/transactions
- core/Engine/messengers/transaction-controller-messenger
Lines changed: 43 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
342 | 342 | | |
343 | 343 | | |
344 | 344 | | |
| 345 | + | |
345 | 346 | | |
346 | 347 | | |
347 | 348 | | |
| |||
380 | 381 | | |
381 | 382 | | |
382 | 383 | | |
| 384 | + | |
| 385 | + | |
| 386 | + | |
| 387 | + | |
| 388 | + | |
| 389 | + | |
| 390 | + | |
| 391 | + | |
| 392 | + | |
| 393 | + | |
| 394 | + | |
| 395 | + | |
| 396 | + | |
| 397 | + | |
| 398 | + | |
| 399 | + | |
| 400 | + | |
| 401 | + | |
| 402 | + | |
383 | 403 | | |
384 | 404 | | |
385 | 405 | | |
| |||
438 | 458 | | |
439 | 459 | | |
440 | 460 | | |
| 461 | + | |
| 462 | + | |
| 463 | + | |
| 464 | + | |
| 465 | + | |
| 466 | + | |
| 467 | + | |
| 468 | + | |
| 469 | + | |
| 470 | + | |
| 471 | + | |
| 472 | + | |
| 473 | + | |
| 474 | + | |
| 475 | + | |
| 476 | + | |
| 477 | + | |
| 478 | + | |
| 479 | + | |
| 480 | + | |
| 481 | + | |
| 482 | + | |
| 483 | + | |
441 | 484 | | |
442 | 485 | | |
Lines changed: 6 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
36 | | - | |
| 36 | + | |
| 37 | + | |
37 | 38 | | |
38 | 39 | | |
39 | 40 | | |
| |||
49 | 50 | | |
50 | 51 | | |
51 | 52 | | |
52 | | - | |
| 53 | + | |
53 | 54 | | |
54 | 55 | | |
55 | 56 | | |
| |||
76 | 77 | | |
77 | 78 | | |
78 | 79 | | |
| 80 | + | |
79 | 81 | | |
80 | 82 | | |
81 | 83 | | |
82 | 84 | | |
83 | 85 | | |
84 | 86 | | |
85 | 87 | | |
86 | | - | |
| 88 | + | |
87 | 89 | | |
88 | 90 | | |
89 | 91 | | |
| |||
92 | 94 | | |
93 | 95 | | |
94 | 96 | | |
| 97 | + | |
95 | 98 | | |
96 | 99 | | |
97 | 100 | | |
| |||
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
148 | 148 | | |
149 | 149 | | |
150 | 150 | | |
| 151 | + | |
151 | 152 | | |
152 | 153 | | |
153 | 154 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
176 | 176 | | |
177 | 177 | | |
178 | 178 | | |
179 | | - | |
| 179 | + | |
180 | 180 | | |
181 | 181 | | |
182 | 182 | | |
| |||
286 | 286 | | |
287 | 287 | | |
288 | 288 | | |
289 | | - | |
290 | | - | |
| 289 | + | |
| 290 | + | |
291 | 291 | | |
292 | 292 | | |
293 | 293 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7408 | 7408 | | |
7409 | 7409 | | |
7410 | 7410 | | |
| 7411 | + | |
| 7412 | + | |
| 7413 | + | |
| 7414 | + | |
| 7415 | + | |
| 7416 | + | |
| 7417 | + | |
| 7418 | + | |
| 7419 | + | |
| 7420 | + | |
| 7421 | + | |
| 7422 | + | |
| 7423 | + | |
| 7424 | + | |
| 7425 | + | |
| 7426 | + | |
| 7427 | + | |
| 7428 | + | |
| 7429 | + | |
| 7430 | + | |
| 7431 | + | |
| 7432 | + | |
| 7433 | + | |
| 7434 | + | |
| 7435 | + | |
| 7436 | + | |
| 7437 | + | |
| 7438 | + | |
| 7439 | + | |
| 7440 | + | |
| 7441 | + | |
| 7442 | + | |
| 7443 | + | |
| 7444 | + | |
| 7445 | + | |
| 7446 | + | |
| 7447 | + | |
| 7448 | + | |
| 7449 | + | |
| 7450 | + | |
| 7451 | + | |
| 7452 | + | |
| 7453 | + | |
| 7454 | + | |
| 7455 | + | |
| 7456 | + | |
| 7457 | + | |
| 7458 | + | |
| 7459 | + | |
| 7460 | + | |
| 7461 | + | |
| 7462 | + | |
7411 | 7463 | | |
7412 | 7464 | | |
7413 | 7465 | | |
| |||
7561 | 7613 | | |
7562 | 7614 | | |
7563 | 7615 | | |
| 7616 | + | |
| 7617 | + | |
| 7618 | + | |
| 7619 | + | |
| 7620 | + | |
| 7621 | + | |
| 7622 | + | |
| 7623 | + | |
| 7624 | + | |
| 7625 | + | |
| 7626 | + | |
| 7627 | + | |
| 7628 | + | |
| 7629 | + | |
| 7630 | + | |
| 7631 | + | |
| 7632 | + | |
| 7633 | + | |
| 7634 | + | |
| 7635 | + | |
| 7636 | + | |
| 7637 | + | |
| 7638 | + | |
| 7639 | + | |
| 7640 | + | |
| 7641 | + | |
| 7642 | + | |
| 7643 | + | |
| 7644 | + | |
| 7645 | + | |
| 7646 | + | |
| 7647 | + | |
7564 | 7648 | | |
7565 | 7649 | | |
7566 | 7650 | | |
| |||
7583 | 7667 | | |
7584 | 7668 | | |
7585 | 7669 | | |
| 7670 | + | |
| 7671 | + | |
| 7672 | + | |
| 7673 | + | |
| 7674 | + | |
| 7675 | + | |
| 7676 | + | |
| 7677 | + | |
| 7678 | + | |
| 7679 | + | |
| 7680 | + | |
| 7681 | + | |
| 7682 | + | |
| 7683 | + | |
| 7684 | + | |
| 7685 | + | |
| 7686 | + | |
| 7687 | + | |
| 7688 | + | |
| 7689 | + | |
| 7690 | + | |
| 7691 | + | |
7586 | 7692 | | |
7587 | 7693 | | |
7588 | 7694 | | |
| |||
8553 | 8659 | | |
8554 | 8660 | | |
8555 | 8661 | | |
| 8662 | + | |
| 8663 | + | |
| 8664 | + | |
| 8665 | + | |
| 8666 | + | |
| 8667 | + | |
| 8668 | + | |
| 8669 | + | |
| 8670 | + | |
| 8671 | + | |
| 8672 | + | |
| 8673 | + | |
| 8674 | + | |
| 8675 | + | |
| 8676 | + | |
| 8677 | + | |
| 8678 | + | |
| 8679 | + | |
| 8680 | + | |
| 8681 | + | |
8556 | 8682 | | |
8557 | 8683 | | |
8558 | 8684 | | |
| |||
9437 | 9563 | | |
9438 | 9564 | | |
9439 | 9565 | | |
9440 | | - | |
9441 | | - | |
9442 | | - | |
| 9566 | + | |
| 9567 | + | |
| 9568 | + | |
9443 | 9569 | | |
9444 | 9570 | | |
9445 | 9571 | | |
| |||
9471 | 9597 | | |
9472 | 9598 | | |
9473 | 9599 | | |
9474 | | - | |
| 9600 | + | |
9475 | 9601 | | |
9476 | 9602 | | |
9477 | 9603 | | |
| |||
9513 | 9639 | | |
9514 | 9640 | | |
9515 | 9641 | | |
9516 | | - | |
9517 | | - | |
9518 | | - | |
| 9642 | + | |
| 9643 | + | |
| 9644 | + | |
9519 | 9645 | | |
9520 | 9646 | | |
9521 | 9647 | | |
| |||
9547 | 9673 | | |
9548 | 9674 | | |
9549 | 9675 | | |
9550 | | - | |
| 9676 | + | |
9551 | 9677 | | |
9552 | 9678 | | |
9553 | 9679 | | |
9554 | | - | |
9555 | | - | |
9556 | | - | |
| 9680 | + | |
| 9681 | + | |
| 9682 | + | |
9557 | 9683 | | |
9558 | 9684 | | |
9559 | 9685 | | |
| 9686 | + | |
9560 | 9687 | | |
| 9688 | + | |
| 9689 | + | |
9561 | 9690 | | |
| 9691 | + | |
9562 | 9692 | | |
9563 | 9693 | | |
| 9694 | + | |
| 9695 | + | |
| 9696 | + | |
9564 | 9697 | | |
9565 | 9698 | | |
9566 | 9699 | | |
9567 | 9700 | | |
9568 | 9701 | | |
9569 | | - | |
9570 | | - | |
9571 | | - | |
9572 | | - | |
9573 | | - | |
9574 | | - | |
9575 | | - | |
9576 | | - | |
9577 | | - | |
| 9702 | + | |
9578 | 9703 | | |
9579 | 9704 | | |
9580 | 9705 | | |
| |||
35612 | 35737 | | |
35613 | 35738 | | |
35614 | 35739 | | |
35615 | | - | |
35616 | | - | |
| 35740 | + | |
| 35741 | + | |
35617 | 35742 | | |
35618 | 35743 | | |
35619 | 35744 | | |
| |||
0 commit comments