Commit Graph

3169 Commits

Author SHA1 Message Date
Daniel Sanchez 5e5736827f
Merge pull request #1807 from gnosis/hotfix/refund-displayed-as-rejection
Mergeback hotfix v2.18.1 to development
2021-01-25 16:43:58 +01:00
Daniel Sanchez 3748245a79
Merge branch 'development' into hotfix/refund-displayed-as-rejection 2021-01-22 19:49:50 +01:00
Daniel Sanchez 173054a1b6
Merge pull request #1805 from gnosis/hotix/refund-displayed-as-rejection-desktop-app
Add commit for desktop app v2.17.2
2021-01-22 19:47:38 +01:00
Daniel Sanchez a84577cebb Merge branch 'master' into hotix/refund-displayed-as-rejection-desktop-app 2021-01-22 12:57:28 +01:00
Daniel Sanchez 5bfc4bb15b
Merge pull request #1803 from gnosis/hotfix/refund-displayed-as-rejection
(Hotfix) Refund displayed as transaction rejection
2021-01-22 10:00:08 +01:00
Daniel Sanchez 93f5e8134e Set v2.18.1 2021-01-22 09:04:21 +01:00
Daniel Sanchez f47bc7f175 Set v2.17.2 2021-01-22 09:03:01 +01:00
Agustin Pane a604da1b6c Fix prettier 2021-01-22 08:59:06 +01:00
Agustin Pane b6577f8e6f Updates isCancelTransaction logic and tests 2021-01-22 08:59:06 +01:00
Agustin Pane 67af60ef71 Adds isEmptyAddress method 2021-01-22 08:59:06 +01:00
Agustin Pane 75f1f8829f Fix prettier 2021-01-21 17:46:32 -03:00
Agustin Pane c0d86c42ae Updates isCancelTransaction logic and tests 2021-01-21 15:26:44 -03:00
Agustin Pane 8b1809d8eb Adds isEmptyAddress method 2021-01-21 15:26:25 -03:00
Daniel Sanchez 87470956fc
Merge pull request #1796 from gnosis/release/v2.18.0
Backmerge release v2.18.0 to development
2021-01-19 21:52:30 +01:00
Daniel Sanchez d36e447120
Merge branch 'development' into release/v2.18.0 2021-01-19 19:10:44 +01:00
francovenica 46274b00ec
fix ids (#1755)
add data-testid for sidebar

add id for recipient field

Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
Co-authored-by: Fernando <fernando.greco@gmail.com>
2021-01-19 15:08:51 -03:00
Daniel Sanchez dfc650dd06
Merge branch 'development' into release/v2.18.0 2021-01-19 19:04:28 +01:00
Daniel Sanchez 38c5bdf757
Merge pull request #1794 from gnosis/release/v2.17.1
Release v2.17.1
2021-01-19 19:02:22 +01:00
Daniel Sanchez c0441e97fa Merge branch 'master' into release/v2.17.1 2021-01-19 16:14:55 +01:00
Daniel Sanchez e65849691e Update client signature 2021-01-19 13:17:15 +01:00
Daniel Sanchez 42d438fb53
Merge pull request #1771 from gnosis/release/v2.18.0
Release v2.18.0
2021-01-19 10:06:36 +01:00
Daniel Sanchez d6d820ca3b Update packed files for desktop app 2021-01-19 09:40:30 +01:00
Daniel Sanchez 0e1efc6f8f Set v2.18.0 2021-01-19 09:13:57 +01:00
Agustin Pane f79f37709f
(Feature) Disable gas estimation for WalletConnect (#1790)
* Make calculateGasOf receive object param

* Disable estimation of gas feature for walletConnect

* Add FIXME to walletConnect estimation check

Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
2021-01-18 15:43:05 -03:00
Agustin Pane 0d0acacd04
Adds TransactionFees modal to confirm safe app tx modal (#1789) 2021-01-18 15:25:28 -03:00
Agustin Pane ce43a97bee
(Fix) - #1783 Trezor gas estimation fail (#1788)
* Improves the way we parse the error message on getDataFromNodeErrorMessage for supporting trezor response

* Fix trezor execution not working properly

* Add parameter to processTransaction in ApproveTxModal

Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
2021-01-18 18:04:04 +01:00
Daniel Sanchez 4aa5955b08 Set v2.17.1 2021-01-18 12:07:34 +01:00
Daniel Sanchez 4a10ed045f Build desktop app using latest ubuntu runner 2021-01-18 11:14:29 +01:00
Daniel Sanchez 54ed65ca4e Disable devtools 2021-01-18 10:20:04 +01:00
Daniel Sanchez 6daaf0cb6d Enable dev tools to test electron build 2021-01-16 00:13:17 +01:00
Daniel Sanchez 2264b1cb7e Remove unused blocknative keys
Remove unused build file
2021-01-16 00:12:36 +01:00
Daniel Sanchez b7f1036a60 Disable yarn cache for electron build 2021-01-15 22:47:07 +01:00
Daniel Sanchez 6cc780c994 Upgrade build action to use node 14 2021-01-15 20:17:13 +01:00
Agustín Longoni 0fb1824f49 Fix asset value column alignment (#1778) 2021-01-15 20:12:14 +01:00
Agustín Longoni 188dc80912
Fix asset value column alignment (#1778) 2021-01-15 20:06:05 +01:00
Daniel Sanchez cdbe4dfc57 Cleanup erc20 amount calcs 2021-01-15 17:33:35 +01:00
Agustin Pane 4c45daa2e2 Fix useTxData refactor
Remove unnecessary useTxAmount
2021-01-15 12:35:09 -03:00
Daniel Sanchez 5bcad40c16 Fix electron app image
Update and fix ledger connection for desktop app
2021-01-15 12:04:43 +01:00
Agustin Pane 10f1891f8f
(Fix) - Calculates gas for SpendingLimit transactions (#1773)
* Bug: Use link tag instead of javascript navigation in apps list (#1770)

* Use list instead of programmable navigation

* add declaration for styled-components theme

* (Fix) - Calculates gas for SpendingLimit transactions (#1773)

* Calculates gas for spendingLimit transactions

* Adds TransactionFees component inside UpdateSafeModal

* Fix send collectible gas calculation

Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
2021-01-14 12:36:55 +01:00
Mikhail Mikheev 1a50b59404
Bug: Use link tag instead of javascript navigation in apps list (#1770)
* Use list instead of programmable navigation

* move color style to appcard

* use color from theme

* add declaration for styled-components theme
2021-01-13 17:28:10 +04:00
Agustin Pane 8a774f2e66
(Feature) - #1048 Tx will fail warning (#1675)
* Makes getGasEstimationTxResponse exportable

* Add check for failing txs on approveTxModal

* Adds styles for reviewTx

* Adds useTxSuccessCheck hook

* Adds checkIfTxWillFail function

* Uses checkIfTxWillFailAsync on reviewTx modal

* Improves approveTx modal

* Add check for failing transaction in contract interaction modal

* Add check for reviewCollectible

* Fix check on sendFunds reviewTx

* Adds styling for contractInteraction modal

* Fix gas calculation for native token transfers

* Rename estimateDataGasCosts to parseRequiredTxGasResponse
Adds getPreValidatedSignatures
Refactor estimateTxGasCosts
Refactor checkIfExecTxWillFail

* Refactor checkIfExecTxWill usage

* Refactor checkIfTxWillFailAsync in ReviewTx

* Use getPreValidatedSignatures in createTransaction()

* Refactor estimateTxGasCosts
Rename estimateSafeTxGas to estimateExecTransactionGas

* Refactor ReviewTx: extract useEffects to hooks

* Remove unnecessary gas transfer amount

* Refactor estimateTxGasCosts: extract checkIfTxIsExecution and estimateTxGas

* Fix tx amount
Remove console log

* Moves useCheckIfTransactionWillFail to logic/hooks folder

* Replaces useEffect usage with useCheckIfTransactionWillFail hook
Also fix how some modals fetch the safeAddress

* Improves modal's wording

* Fix error parsing the cancel transaction error message from GETH nodes

* Remove useCheckIfTransactionWillFail
Adds useEstimateTransactionGas
Renames estimateTxGas to estimateTransactionGas
Removes estimateTxGasCosts
Removes checkIfExecTxWillFail

* Replace useCheckIfTransactionWillFail from modals with useEstimateTransactionGas

* Replace estimateGasCosts from every review tx modal with useEstimateTransactionGas

* Replace estimateGasCosts from every review tx modal with useEstimateTransactionGas

* Extract isExecution calculation to useEstimateTransactionGas

* Creates TransactionFailText

* Uses TransactionFailText in the review modals

* Fix wrong selector usage

* Fix missing null check on cancel tx confirmations

* Add guard for CLOSE_SNACKBAR action when tx was already dismissed

* Improves useEstimateTransactionGas in review custom tx and contract interaction review

* Fix review replace/remove/add owner modals styling

* Refactor response of useEstimateTransactionGas

* Remove safeAddress as param to the useEstimateTransactionGas

* Improves how threshold is obtained in useEstimateTransactionGas.tsx

* Rename gasCostHumanReadable to gasCostFormatted

* Add operation to useEstimateTransactionGas

* Refactor ConfirmTransactionModal to use useEstimateTransactionGas

* Refactor proccessTransaction to use getPreValidatedSignatures method

* Fix default export of ApproveTxModal

* Rename estimateExecTransactionGas to estimateGasForTransactionCreation
Remove estimateTransactionGas from gas.ts

* Make estimateGasForTransactionCreation throw error instead of 0 gas

* Adds estimateGasForTransactionExecution and estimateGasForTransactionApproval to gas.ts

* Move estimateTransactionGas to useEstimateTransactionGas
Refactors useEstimateTransactionGas to return isCreation and isOffChainSignature

* Type and refactor generateSignaturesFromTxConfirmations
Moves getPreValidatedSignatures to safeTxSigner.ts

* Uses confirmations to estimateGasForTransactionExecution

* Adds TransactionFeesText component
Uses TransactionFeesText on ApproveTxModal

* Pass more parameters to estimateGasForTransactionExecution

* Removes unnecessary parameter in getNewTxNonce

* Moves checkIfOffChainSignatureIsPossible to safeTxSigner.ts

* Fix check for null confirmations

* Uses checkIfOffChainSignatureIsPossible on createTransaction.ts

* Move TransactionFailText inside TransactionFees component

* Pass safeTxGas to useEstimateTransactionGas.tsx
Improves how we use default params

* Fix gas iteration on estimateGasForTransactionExecution

* Fix estimateGasForTransactionExecution calculation

* Fix generateSignaturesFromTxConfirmations calculation

* Remove unnecessary Promise and await

* Fix estimateGasForTransactionExecution for preApproving owner case

* Improve logging

* Uses operation in useEstimateTransactionGas

Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
Co-authored-by: Fernando <fernando.greco@gmail.com>
2021-01-13 11:27:07 +01:00
Daniel Sanchez 618888ed07
Upgrade dependencies (#1767)
* Deduplicate dependencies

* Upgrade @truffle/contract to latest v4.3.0

* Upgrade dependencies with minor changes

* Upgrade electron-builder configuration

* Upgrade typescript to latest version

* Upgrade safe-apps-sdk to latest version
Remove autoprefixer dependency
2021-01-12 22:33:22 +01:00
Agustín Longoni 31be5b1993
Feature/#1499 add mobile start (#1750)
* add modal, img and card

* add overlay and clean css

* fix breakpoint to 992px (screenMd)

* add mobileView  library

* add link to download app button

* close button styles

* fix close button on hover

* Remove polished rgba for material-ui fade

Co-authored-by: nicolas <nicosampler@users.noreply.github.com>
Co-authored-by: nicosampler <nf.dominguez.87@gmail.com>
Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
2021-01-12 20:26:38 +01:00
Agustin Pane f1ffb5d147
(Fix) - Remove safe fix (#1744)
* Avoid displaying notification for non updated owner

* Remove old selector usage on welcome page

* Remove provider props on WelcomeLayout
Fix removing safe behaviour

* Fix missing undefined property check on cancelThresholdReached

* Removes the default safe when removing the safe marked as default

* Remove index

* Change react import

* Fix relative path for relocate url

* Adds removeLocalSafe action
Uses removeLocalSafe on onRemoveSafeHandler also removes default safe

* Refactor SafeListSidebar uses redux-hooks approach

* Disabled save button when the owner name input is pristine
2021-01-12 11:39:57 -03:00
dependabot[bot] 09b470187f
Bump axios from 0.21.0 to 0.21.1 (#1754)
Bumps [axios](https://github.com/axios/axios) from 0.21.0 to 0.21.1.
- [Release notes](https://github.com/axios/axios/releases)
- [Changelog](https://github.com/axios/axios/blob/v0.21.1/CHANGELOG.md)
- [Commits](https://github.com/axios/axios/compare/v0.21.0...v0.21.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
2021-01-12 14:01:34 +01:00
dependabot[bot] 186fb34731
Bump node-notifier from 8.0.0 to 8.0.1 (#1741)
Bumps [node-notifier](https://github.com/mikaelbr/node-notifier) from 8.0.0 to 8.0.1.
- [Release notes](https://github.com/mikaelbr/node-notifier/releases)
- [Changelog](https://github.com/mikaelbr/node-notifier/blob/v8.0.1/CHANGELOG.md)
- [Commits](https://github.com/mikaelbr/node-notifier/compare/v8.0.0...v8.0.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
2021-01-12 13:02:04 +01:00
Mati Dastugue b7fb6033eb
Merge pull request #1745 from gnosis/fix/balance-table-sort
Fix - Value column not sortable in Assets list
2021-01-05 15:45:02 -03:00
Mati Dastugue 6369d872f8 Merge branch 'fix/balance-table-sort' of github.com:gnosis/safe-react into fix/balance-table-sort 2020-12-28 14:10:38 -03:00
Mati Dastugue a00539e8c2 Fix token price function to be more generic 2020-12-28 14:09:30 -03:00
Mati Dastugue d1cbc1ac8c
Merge branch 'development' into fix/balance-table-sort 2020-12-22 16:19:04 -03:00