Support BOBA L2s #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add tests for tx history
…l#2959) * fix: Hide nonce input for spending limit txs * fix: Display nonce input if user switches from spending limit token to other * fix: reset nonce input depending on type * refactor: Extract hooks from CreateTokenTransfer, add tests * fix: Revert part of the initial fix * fix: lint issue * refactor: Extract read-only address input into AddressBookInput, write more tests * fix: Add tests for new hooks
* Fix: redefine risk wording * Reword again
* feat: add export button to owner list * fix: change button style + export with names
Add more tx history tests
* Feat: queue counter in the sidebar menu * Count recovery attempts as queued txs * Count only transaction items * Show 20+ if there is a next page * Add tests * Add clear mocks
* feat: show wallet balance in header * fix: Style component * fix: Add info to tx creation * fix: Tweak css for dark mode * fix: Move useWallet into a context, add return type to useWalletBalance * fix: Update design of account center * fix: Show wallet balance for social login * fix: Rename wallet balance and show it even if no relaying --------- Co-authored-by: schmanu <[email protected]>
…global#2974) * fix: Disable submit button if there is a high or critical risk * test: Add tests for SignForm * test: Add more tests for SignForm * test: Add happy path tests for ExecuteForm * test: Add more tests for ExecuteForm * refactor: Extract default props in SignForm test * refactor: Only render ApprovalEditor once * test: Add initial SignOrExecuteForm test * test: Add more SignOrExecuteForm tests * test: Implement missing tests, add TODO
* Analytics: include query params in the pageview event * Rm console.log
* feat: Display network logos in network selector * fix: Add alt tag, extract logos * refactor: Extract MenuItem block * fix: Update ChainIndicator with optional prop and display chain logo by default * fix: Reuse ChainIndicator for NetworkSelector, use img instead of Image * test: Update selector for apps_list e2e
* Feat: track tx type on sign/execute * Refactor: centralize tx tracking * Track bulk execution * Add tests * Rm undefined * Track spending limit transfers * Call onSubmit callback w/o waiting for tracking * Swap and rename args * Swap args in getTxDetails * Immediate execution * Fix: track subsequent message signatures as TX_EVENTS.CONFIRM
* Migrate tx builder to sepolia
…fe-global#2981) * Add default values for value and data in AppCommunicator * Set default value for data
* Fix: better labels for relaying * transitionDuration for ConnectionCenter * Fix unit tests
* Fix: remove broken index redirect * Redirect from Welcome to Accounts only on explicit Connect * Adjust smoke test
…afe-global#3310) * feat: add labels to tracking events for the login flow * remove unecessary util function * fix: use correct tracking labels for login_page and sidebar * simplify data import/export events * fix: [RC] Add pay later hint to safe creation step (safe-global#3305) * fix: Turn off copy prefix by default and show toggle in first steps widget * fix: Change 1/1 hint to an info box Show message about Pay later feature Remove unused NetworkWarning from owner step * tests: fix breaking test * fix: Adjust wording for 1/1 hint and pay later hint --------- Co-authored-by: mike10ca <[email protected]> * fix: show newly depoyed safes under owned safes (safe-global#3308) * fix: show newly depoyed safes under owned safes * Update src/components/welcome/MyAccounts/useAllSafes.ts Co-authored-by: katspaugh <[email protected]> * fix: add missing import --------- Co-authored-by: katspaugh <[email protected]> * Show remove from watchlist option when viewing a watchlist safe (safe-global#3307) * Show remove option when safe is already added to the watchlist * remove trash icon from remove button * add tracking for removeing from wtchlist * fix: remove duplicate event * fix: typo in remove from watchlist event * fix: add missing labels and events * fix: incorrect label for deleted from watchlist event --------- Co-authored-by: Usame Algan <[email protected]> Co-authored-by: mike10ca <[email protected]> Co-authored-by: katspaugh <[email protected]>
* tests: add revised tests for import export * tests: remove unused import * tests: update test logic
Since the .listItem had a padding, the <a> tag inside of it was only clickable if you manage to hit it directly. But ListItem actually looks like a clickable button, so you won’t expect that you have to be clicking directly in the center.
…nts page (safe-global#3376) * index route redirects to login page if there are safes stored locally * fix: update unit tests for WalletLogin
* fix - token icon not refreshing When navigating to a token that doesn’t have an icon we were failing to update the icon when the user navigate to a token that has one. * fix: don’t show token name on new line if too long * Update src/components/tx-flow/flows/TokenTransfer/CreateTokenTransfer.tsx Co-authored-by: katspaugh <[email protected]> --------- Co-authored-by: katspaugh <[email protected]>
…s with watchlist safes (safe-global#3385) * add View Watchlist button to welcome page when there are safes to show * Update src/components/welcome/WelcomeLogin/index.tsx Co-authored-by: katspaugh <[email protected]> --------- Co-authored-by: katspaugh <[email protected]>
* rename owner to signer * fix: smoke tests * fix: change an signer to a signer * revert formatting changes * fix: name of signers export file
…fe-global#3368) * fix: Don't show native transfer for unknown contract interactions * fix: Additionally check for empty data
* show all safes after clicking show more * remove show more button
- Fixes gas estimation for execution transaction - Fixes gas estimation for on-chain signature transaction Co-authored-by: schmanu <[email protected]>
* tests: add happy path flows for recovery v1 * tests: remove unused import * tests: fix failing tests * tests: update test folders * tests: update tests and balance fixture file
… non-owners (safe-global#3398) * fix: Track Create new Safe click once * fix: Disable Add funds and Create transaction buttons for non-owners
- fix: detect native transfers - add unit tests for decoded tx
* remove dynamic sorting by current chain for safe lists * fix: useMemo dependency array * fix: remove unused variable
Branch preview⏳ Deploying a preview site... |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What it solves
Resolves #
How this PR fixes it
How to test it
Screenshots
Checklist