[Epic] Tx flow redesign #2067

katspaugh · 2023-06-02T06:22:35Z

Resolves #1828.

* feat: Add StepperFactory, new tx pages * wip: context instead of routes * Add tx flows to new modal context * fix: Close modal when user navigates * Export all flows from one file * refactor: Remove unused code from StepperFactory * fix: Duplicate ModalDialog to fix failing tests

* Refactor: reactive recommended nonce Reduce re-renders Nonce -1 * Use "nonceReadyOnly" instead of -1 * Split sign and execute into two components

* feat: add tx status widget UI * add logic to paint the stepper * port the ConfirmProposedTx to the new layout * pass tx data to the TxStatusWidget to get the tx confirmations * style: add new lines between adjacent tags

* Refactor: SafeTxProvider * Avoid oscillating nonce * fix reject tx for new provider --------- Co-authored-by: Usame Algan <[email protected]>

* Refactor: useRecommendedNonce + useSafeTxGas * Fix lint * Get recommended nonce by "estimating" an empty tx

github-actions · 2023-06-02T06:25:23Z

Branch preview

✅ Deploy successful!

https://epic_tx_flow--walletweb.review-wallet-web.5afe.dev

github-actions · 2023-06-02T06:26:14Z

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	0	0
Ignored	0	N/A

Result: ✅ success
Annotations: 0 total

^{Report generated by eslint-plus-action}

* Refactor: a less verbose ModalProvider * Fix Cannot redefine property: dispatchTxProposal

* Refactor the tx flow system * Array

* Editable nonce * Fix tests

* fix: Close modal on sign, only render current step * fix: Close modal on sign, only render current step * fix: set all data + move back button --------- Co-authored-by: Usame Algan <[email protected]>

* Refactor: single type for tx flows * Omit txNonce * FormData type

* feat: Port change threshold flow * fix: Only estimate gas when executing * fix: Remove old component

* feat: add owner flow * feat: replace owner flow * feat: replace owner flow

* fix: Show recommended nonce in list * fix: button ref issue

* fix: Check execute checkbox if execution is the only option (#2230) * v1.14.0 * Chore: escape quotes in release action (#2236) Use printf Use a heredoc Use jq * fix: only initiate WC v2 module if project id provided (#2228) * chore: mark `NEXT_PUBLIC_WC_PROJECT_ID` required * chore: don't prettify * fix: don't return module * fix: revert readme * fix: autoconnect to WC v2 on mobile (#2238) * fix: update token price tooltip (#2257) --------- Co-authored-by: katspaugh <[email protected]> Co-authored-by: katspaugh <[email protected]> Co-authored-by: Aaron Cook <[email protected]>

github-actions · 2023-07-10T16:27:08Z

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	0	0
Ignored	0	N/A

Result: ✅ success
Annotations: 0 total

^{Report generated by eslint-plus-action}

* fix: Hide status for batch execute * fix: Close batch execute flow when submitting tx * fix: Remove hide status flag, add isBatch flag and display different status messages for batch execution

* fix: Hide approval editor for non-approval transactions * fix: Adjust useApprovalInfos hook for non-approval txs

* fix: increase `min-width` of nonce field * fix: prevent recommended nonce jump

* fix: Align decoded tx ui for batch execute * fix: Don't show confirmation dialog for batch execute, update spending limit components to use SendAmountBlock

* fix: Validation for spending limits * fix: Reset amount field to empty string instead of 0 when switching tokens * fix: Add readonly addressbook input to spending limits

* fix: Hide nonce for spending limit txs * Update src/components/tx-flow/flows/TokenTransfer/ReviewSpendingLimitTx.tsx Co-authored-by: Aaron Cook <[email protected]> --------- Co-authored-by: Aaron Cook <[email protected]>

usame-algan and others added 10 commits May 23, 2023 12:04

Add folder structure for tx flow

00f0e11

feat: create new-tx route and NewTxMenu component

92efde1

Refactor: reactive recommended nonce (#2050)

6387992

* Refactor: reactive recommended nonce Reduce re-renders Nonce -1 * Use "nonceReadyOnly" instead of -1 * Split sign and execute into two components

Merge dev

a2a2933

feat: Transaction status widget (#2062)

50ab68c

* feat: add tx status widget UI * add logic to paint the stepper * port the ConfirmProposedTx to the new layout * pass tx data to the TxStatusWidget to get the tx confirmations * style: add new lines between adjacent tags

Refactor: SafeTxProvider (#2064)

5e8476d

* Refactor: SafeTxProvider * Avoid oscillating nonce * fix reject tx for new provider --------- Co-authored-by: Usame Algan <[email protected]>

Move progress to TxLayout

a4c362c

Refactor: useRecommendedNonce + useSafeTxGas (#2066)

1152232

* Refactor: useRecommendedNonce + useSafeTxGas * Fix lint * Get recommended nonce by "estimating" an empty tx

Merge dev

50f58d7

katspaugh and others added 10 commits June 2, 2023 08:49

Adjust ConfirmTx and ExecuteTxButton

72a126d

Refactor: a less verbose ModalProvider (#2068)

603a6d2

* Refactor: a less verbose ModalProvider * Fix Cannot redefine property: dispatchTxProposal

Refactor the tx flow system (#2071)

0bd24db

* Refactor the tx flow system * Array

Fix: pass nonce correctly (#2072)

c810ad3

Fix linting errors, add TODOs for failing tx flows (#2070)

8d74bfc

Merge dev

61cf1b5

Editable nonce (#2077)

1362202

* Editable nonce * Fix tests

refactor: save entire data structure on submission (#2083)

1b63a8f

* fix: Close modal on sign, only render current step * fix: Close modal on sign, only render current step * fix: set all data + move back button --------- Co-authored-by: Usame Algan <[email protected]>

feat: Add SuccessScreen to tx flow (#2080)

6f51b42

Refactor: single type for tx flows (#2084)

2e8a049

* Refactor: single type for tx flows * Omit txNonce * FormData type

usame-algan mentioned this pull request Jun 13, 2023

feat: Change threshold flow #2085

Merged

3 tasks

iamacook and others added 5 commits June 13, 2023 12:35

feat: add owner flow (#2086)

b966ab5

feat: Change threshold flow (#2085)

eb1731e

* feat: Port change threshold flow * fix: Only estimate gas when executing * fix: Remove old component

feat: remove owner flow

3afdea1

feat: replace owner flow (#2087)

f0c63ef

* feat: add owner flow * feat: replace owner flow * feat: replace owner flow

feat: update flow (#2089)

5b5c261

usame-algan mentioned this pull request Jun 13, 2023

Remove module flow #2090

Merged

3 tasks

fix: Remove Module tx flow (#2090)

f4c6b86

fix: Show recommended nonce in list (#2260)

878032e

* fix: Show recommended nonce in list * fix: button ref issue

This was referenced Jul 10, 2023

fix: Disable submit button for non-owners #2261

Merged

fix: Hide approval editor for non-approval transactions #2262

Merged

fix: Review send NFT view full width #2263

Merged

usame-algan and others added 4 commits July 10, 2023 17:52

fix: Disable submit button for non-owners (#2261)

3e56abd

fix: Review send NFT view (#2263)

089e287

Merge branch 'dev' into epic-tx-flow

db76669

This was referenced Jul 11, 2023

[TX flow] Batch execute fixes #2268

Merged

[TX flow] Sticky close button #2269

Merged

usame-algan and others added 4 commits July 11, 2023 14:55

[TX flow] Batch execute fixes (#2268)

6d6859e

* fix: Hide status for batch execute * fix: Close batch execute flow when submitting tx * fix: Remove hide status flag, add isBatch flag and display different status messages for batch execution

fix: remove unnecessary hook (#2271)

21878ba

fix: Sticky close button (#2269)

57c053f

fix: Hide approval editor for non-approval transactions (#2262)

9792587

* fix: Hide approval editor for non-approval transactions * fix: Adjust useApprovalInfos hook for non-approval txs

usame-algan mentioned this pull request Jul 11, 2023

fix: Adjust tx flow elements height #2272

Merged

3 tasks

iamacook and others added 3 commits July 11, 2023 16:32

Merge branch 'dev' into epic-tx-flow

cbf8119

fix: Adjust tx flow elements height (#2272)

62c740d

fix: increase min-width of nonce field (#2270)

da39b3d

* fix: increase `min-width` of nonce field * fix: prevent recommended nonce jump

usame-algan mentioned this pull request Jul 11, 2023

fix: Align decoded tx ui for batch execute #2275

Merged

3 tasks

iamacook and others added 2 commits July 11, 2023 17:09

fix: align progress bar with new tx button (#2274)

d2113f3

fix: Align decoded tx ui for batch execute (#2275)

f487649

* fix: Align decoded tx ui for batch execute * fix: Don't show confirmation dialog for batch execute, update spending limit components to use SendAmountBlock

usame-algan mentioned this pull request Jul 12, 2023

fix: Validation for spending limits #2277

Merged

3 tasks

fix: Validation for spending limits (#2277)

cbb24e3

* fix: Validation for spending limits * fix: Reset amount field to empty string instead of 0 when switching tokens * fix: Add readonly addressbook input to spending limits

usame-algan mentioned this pull request Jul 12, 2023

fix: Hide nonce for spending limit txs #2278

Merged

3 tasks

fix: Hide nonce for spending limit txs (#2278)

5ceda2e

* fix: Hide nonce for spending limit txs * Update src/components/tx-flow/flows/TokenTransfer/ReviewSpendingLimitTx.tsx Co-authored-by: Aaron Cook <[email protected]> --------- Co-authored-by: Aaron Cook <[email protected]>

usame-algan approved these changes Jul 12, 2023

View reviewed changes

iamacook approved these changes Jul 12, 2023

View reviewed changes

iamacook merged commit 93b761e into dev Jul 12, 2023
6 of 7 checks passed

github-actions bot locked and limited conversation to collaborators Jul 12, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Epic] Tx flow redesign #2067

[Epic] Tx flow redesign #2067

katspaugh commented Jun 2, 2023 •

edited

Loading

github-actions bot commented Jun 2, 2023 •

edited

Loading

github-actions bot commented Jun 2, 2023 •

edited

Loading

github-actions bot commented Jul 10, 2023 •

edited

Loading

[Epic] Tx flow redesign #2067

[Epic] Tx flow redesign #2067

Conversation

katspaugh commented Jun 2, 2023 • edited Loading

github-actions bot commented Jun 2, 2023 • edited Loading

Branch preview

github-actions bot commented Jun 2, 2023 • edited Loading

ESLint Summary View Full Report

github-actions bot commented Jul 10, 2023 • edited Loading

ESLint Summary View Full Report

katspaugh commented Jun 2, 2023 •

edited

Loading

github-actions bot commented Jun 2, 2023 •

edited

Loading

github-actions bot commented Jun 2, 2023 •

edited

Loading

github-actions bot commented Jul 10, 2023 •

edited

Loading