Bannik rebase dev #6
Open
Conversation
This file contains hidden or 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
Signed-off-by: Chawye Hsu <[email protected]>
* Fix: strip query params from Sentry events * Allow appUrl
* Fix: sort Safe App tags * Remember view mode * Rm categories + add tracking * Fix list view card height
* fix: support tokens with no decimals * fix: decimal length validation
* feat: buttons to expand/collapse all actions * fix: set state in `useEffect` * fix: improve clarity of code
* fix: Delay switching networks until tx execution * refactor: Use onboard instead of wallet to switch network * fix: Return updated wallet when switching, reorder function params * fix: useIsValidTransaction with patched provider, fix tests * fix: create safe e2e test * fix: Invert condition for patched signer * fix: Show gasLimit error for custom gasLimits, switch from using the sdk for useIsValidExecution * fix: Add slight timeout when switching wallet network * fix: Failing test * fix: Only add prevalidated signature if wallet is owner * fix: Remove useIsWrongChain and adjust places where it was used * fix: Remove ChainSwitcher * fix: Address feedback * fix: tests * fix: don't error when switch hardware wallet chain * fix: throw error for incorrect chain * fix: don't throw inside promise * fix: Safe Apps dispatcher * fix: keep header switcher + rename function * feat: show warning for hardware wallets * refactor: remove `sleep` * fix: mocks + remove log * fix: off-chain sign switch + reconnect SC wallets * fix: move warning to alert + reduce PR size * fix: tests + reduce PR size * fix: enable submit + init SDK when addresses match * fix: Remove unused dependencies, remove useIsGranted hook, add warning to batch execute and spending limit txs * fix: ensure styles are important --------- Co-authored-by: iamacook <[email protected]>
* feat: display Fallback Handler address * feat: display name, adjust tooltip + test * fix: use `logoUri`
* feat: show alert for new versions in settings * fix: add loading skeletons * fix: design of info box
* fix: Add licenses page * chore: Change safe-modules-deployments package source * chore: Remove dev dependencies, make table columns same width
* fix: Dismiss update safe notification for 90 days * fix: Remove middleware, use hashmap to store dismissals * fix: Revert notificationSlice changes * fix: use useLocalStorage * fix: Use onClose handler for notification * fix: failing tests, remove exports, move type * fix: Move outdated safe check after dismissNotification check for cleanup
* feat: relay transaction execution modal (safe-global#1724) * feat: ExecutionMethod component * add New chip * add useRemainingRelays hook * add better error to useRemainingRelays * willRelay styles * fix prop type * move styles to css modules * refactor hook * change state variable type * change prop type * fix condition to show <ExecutionMethod> * simplify redundant expression * feat: relay modal adv params (safe-global#1741) * feat: disable uneditable advanced param fields * style: relay limit reached * feat: relay a transaction trough the `sponsoredCall` endpoint (safe-global#1730) * feat: ExecutionMethod component * add New chip * add useRemainingRelays hook * add better error to useRemainingRelays * willRelay styles * fix prop type * move styles to css modules * refactor hook * change state variable type * change prop type * fix condition to show <ExecutionMethod> * feat: executeSponsoredCall and useSponsoredCall * feat: sign txs before proposing and relaying * WIP: pending status for relayed txs * fix: resubscribe to waitForRelayTx on page load * refactor onRelay * fix jest tests * move call data signatures encoding to the dispatcher * do not pass redundant param * set SafeGelato relay service url as env var * get canRelay from chain features * extract to a const the initial polling delay * create enum for task states * display execution method if wallet type can relay * remove gelato-sdk dep from txMonitor. tests * include v1 path in SAFE_GELATO_RELAY_SERVICE_URL constant * add tests to SignOrExecuteForm * add timeout to waitForRelayedTx * improve unit tests * refactor: use useAsync in useWalletCanRelay * minor comments * refactor: abstract hasEnoughtSignatures --------- Co-authored-by: schmanu <[email protected]> * fix: use gelato relay service url as base url * fix: clear relay timeout timer on success / errors * fix: pass new env variable to build action * revert: expected relay service url * feat: Relay dashboard widget (safe-global#1787) * feat: add dashboard Relay widget * conditionally display remaining transactions * rm redundant attributes * Update src/components/dashboard/Relaying/index.tsx Co-authored-by: Aaron Cook <[email protected]> * styles: fix styles as per design * style: adjust relaying icon * test: unit test useRemainingRelays * Update src/components/dashboard/Relaying/index.tsx Co-authored-by: Aaron Cook <[email protected]> * style: fix chip text color --------- Co-authored-by: Aaron Cook <[email protected]> * feat: track relaying events (safe-global#1798) * add tracking to tx execution methods * add tracking to the help article link * refactor: Relay UI changes (safe-global#1800) * feat: update dashboard widget * add SponsoredBy component * remove ExecutionMethod component * fix unit tests * style: increase space around divider * fix dashboard widget style * address PR comments * fix "status not found" error message * remove unnecessary element * fix: monitoring of relayed txs * fix: SignOrExecute jest test * feat: Relay safe creation (safe-global#1815) * add SponsoredBy component to the create Safe review step * wip relay create safe * do not propose multiple times * add relay task monitoring * fill Validating step if taskid * handling relay error * add useLeastRemainingRelays hook * refactor and tests * rm unnecessary code * improve review step UI * check if chain has relaying feature before the network request * fix wrong loading expression * refactor txMonitor Co-authored-by: Aaron Cook <[email protected]> * refactor: use safe deployments * rm console log * refactor: `break` from `switch` to clear --------- Co-authored-by: schmanu <[email protected]> Co-authored-by: Aaron Cook <[email protected]> Co-authored-by: iamacook <[email protected]> * feat: relay batch modal (safe-global#1820) * feat: add SponsoredBy and relay condition to batched txs modal * wip dispatch batch execution relay * encode multisend call data * feat: polling for batched relayed txs * add tooltip * test: dispatchBatchExecutionRelay * rm redundant code * pr comments * add fetchRemainingRelays return type --------- Co-authored-by: schmanu <[email protected]> * refactor: split create safe callbacks (safe-global#1824) * chore: update readme (safe-global#1826) * fix: env-relative relay URL (safe-global#1832) * fix: cleanup relay env. vars (safe-global#1833) * fix: cleanup relay env. vars * fix: add relay URLs from secrets * fix: take `localStorage` flag into account * test: e2e relayer (safe-global#1797) * test relayer * fix prettier * allow tx queueing * searching by labels * moving test to smoke folder * change goerli expression * fix tx proposal validation * test: update the e2e create_tx per latest relay designs --------- Co-authored-by: Diogo Soares <[email protected]> --------- Co-authored-by: Diogo Soares <[email protected]> Co-authored-by: schmanu <[email protected]> Co-authored-by: Diogo Soares <[email protected]> Co-authored-by: Aaron Cook <[email protected]> Co-authored-by: francovenica <[email protected]>
…-global#1846) * Fix: remove atomic0 * Add Zerion links
* refactor: remove `getWeb3` * refactor: remove unnecessary `web3` instance * fix: update relayer
* fix: Track most searched safe apps * fix: Increase search debounce to 2s
New component in transaction modals if approval txs were found. Offers a way for the user to adjust the approval amount(s) before submitting the transaction.
…l#1865) * fix: Align header actions on mobile * fix: Connect wallet word wrap on mobile * fix: Remove sticky from tx filter form on mobile * fix: Address feedback
* fix: Unhandled exception in useOnboard * fix: added logger message and added test case for error in getAddress * Update src/hooks/wallets/useOnboard.test.ts Co-authored-by: Aaron Cook <[email protected]> --------- Co-authored-by: Aaron Cook <[email protected]>
* fix: limit `origin` length * fix: prioritise URL * fix: add test
* Fix: tx can have been created w/o signatures * Reuse the same logic in ExecuteForm
* Fix: cookie prefs checked checkbox color * Move colors to CSS * Stop propagation on ExplorerButton in EthHashInfo
…al#2358) * Fix: prevent an early redirect on the Safe Apps share page * Use an effect
* fix: Use PNG instead of SVG for Redefine logo * fix: Optimize PNG
* Refactor: fetch pending queue only in 1/N Safes * Improve hook condition * Fix previous nonces
Branch preview❌ Deploy failed! |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Analytics changes
Screenshots