* fix: Send funds not working when selecting receipt from addressBook
Also, this commit includes an intent to unify/simplify SendModal component
fixes#632
* Set default value to txData for custom txs
fixes#632
The fix attempts to properly differentiate an ERC-721 from an ERC-20 token transaction by identifying if it's a `transfer` transaction looking for a `decimals` method in its code. It the later is not found, then it's considered an ERC-721.
fixes#678
* Add Assets sections
* (add) collectibles tab
* (add) criptokitty items
* (add) collectible items, definitive edition
* (fix) collectibles were overlapping with bottom banner
* (fix) wording
* (fix) responsive issues
* Install `async-sema` dependency
* Create collectible source classes
- source from mocked data and opensea, it's extendable to import information from other sources
* Update `Collectible` implementation to use new data source
* Create constants file to better handle env variables and default values
* Add description to item's cards
- also added a mocked class with real data
* Fix `saveTxToHistory`, remove hardcoded `CALL`
* Fix after merge development
* Set background color for collectible based on data info
- Changed `withStyles` in favor of a hook-like approach with `makeStyles`
* Enhance collectible card info and group title
* Use current safeAddress to query for collectibles information
- also migrated from `withStyles` to `makeStyles`
* Use proper key values for lists and set more significant names
* update yarn.lock after merge
* Fix linting error
* Move ethAsToken verification outside loop
* Use absolute route for `SendModal` import
* Move Collectibles into redux store
* Update yarn.lock file
* Selectable NFTs
* Divide the `collectible` store into `nftAssets` and `nftTokens`
- Also updated components to retain functionality
- Created a `textShortener` function for better presentation
* Update `yarn.lock`
* Update `yarn.lock`
* Fix item background color
* Clears the tokenID select field when the collectible selected changes
* Open Send modal from the assets section
* Use token name for the token selection dropdown
* Add openZeppelin contracts dependency
* Create ERC721 getter
* Fix types, default values and clean code
* Fix: properly refresh list of collectibles when switching safes
* Add ReviewCollectible step in send NFT
* Change items shadow
* Give option to choose what to send by clicking 'Send' button in AddressBook
* Disable [Send] button for Collectibles if not owner
* Set Coins as default option in assets tab
- also fixed styles for `Coins` option
* Use collectible icon in send modal
* Set default message when no assets available
- removed pagination feature
* Create SafeVersionProvider to better handle version-related tasks
Provides:
- current and latest versions,
- a boolean indicating a need for update,
- an upgradeSafe callback to trigger upgrade from any place,
- a list of enabled features, depending on the current version
- the latter needs a refactor like extract features outside the provider
and define constants for the features.
* Force build
* Update `yarn.lock`
* Disable Manage list for NFTs
* Fix container shadow
- Also fixes tables shadow, thanks to @gabitoesmiapodo
* Enable nested routes for balances (assets) tab
* Default to `/balance` if invalid nested path
* Disable [Send Collectible] button, if not supported by safe
* Change sub-menu buttons to clickable text
* Replace Paragraph with Link
* Fix invalid props errors for Link component
* Fallback to `transferFrom` if `safeTransferFrom` is not implemented
* Use `transfer` as fallback to ERC-721's `safeTransferFrom`
- need to identify ERC721 token using `transfer` and `name` methods
* Display failed transactions
* Use react.lazy for collectibles' modals
* Identify ERC-721 token transaction
* Fix Send Collectibles modal layout/behavior
- disable dropdown list if there's no item to pick
- fix placeholder for tokens list
- fix dropdown list styles
* Set default `isSuccessful` flag to `true`
* Save version related values into store
- each safe has its `currentVersion`, `needsUpdate` and `featuresEnabled`
- and safes store has the `latestMasterContractVersion`
* Migrate Balance to use store-provided values
* Migrate Settings to use store-provided values
* Migrate ChooseTxType to use store-provided values
* Remove SafeVersionProvider
Co-authored-by: Gabriel Rodriguez Alsina <gabriel.rodriguez@altoros.com>
Co-authored-by: apane <agustin.pane@gmail.com>
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
* Bug: #482 - Address book Custom transactions Recipient validation (#577)
* (fix) text input bottom border
* Fix#482 input
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# src/components/forms/TextField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/AddressBookInput/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/TokenSelectField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/index.jsx
# yarn.lock
* Fix custom tx addresses filtering
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# yarn.lock
* Remove console logs
Fixed prettier issues
* Remove unnecessary template string
* Fix `tokenAddress` string conversion
* Use `secondaryBackground` value
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (feature) Adding origin for Apps Transactions (#576)
* Adding origin field when creates a TX
* refactor: replace list of arg by object in getApprovalTransaction and getExecutionTransaction function
* minor changes
* Allow execute if threshold is 1 for the first tx
- Related to issue #563
- `lastTx` is required due to #489
* - Normalizing logic between createTransaction and processTransaction
- Moving shared function to a new file
* Refactor `doesTxNeedApproval` back to the `isExecution`-related meaning
* Rename function and variable names
* Add tests for `getNewTxNonce` and `shouldExecuteTransaction` functions
* Pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
* Update Tests
- remove mocked `getGnosisSafeInstanceAt`
- pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (Feature) Add checks into CI (#594)
* Repare new dev and staging enviroments to deploy (#618)
* Repare new dev and staging enviroments to deploy
- Add to `deploy_pull_requests.sh` to the new path.
- Replace upload-dir to deploy staging to the new path.
* add /app/ public path for the webapp, remove duplicate yarn build from travis
* add basename to app
* deploy pr to /app
* update travis
* linter error fixes
* fix travis build script
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
* (fix) sidebar contents' scrolling (#622)
* (add) linter sorting rules (#614)
* (add) linter sorting rules
* (fix) linting errors according to the new rules
Co-authored-by: Fernando <fernando.greco@gmail.com>
* Rebase master onto development (#624)
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
* Bug: #482 - Address book Custom transactions Recipient validation (#577)
* (fix) text input bottom border
* Fix#482 input
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# src/components/forms/TextField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/AddressBookInput/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/TokenSelectField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/index.jsx
# yarn.lock
* Fix custom tx addresses filtering
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# yarn.lock
* Remove console logs
Fixed prettier issues
* Remove unnecessary template string
* Fix `tokenAddress` string conversion
* Use `secondaryBackground` value
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (feature) Adding origin for Apps Transactions (#576)
* Adding origin field when creates a TX
* refactor: replace list of arg by object in getApprovalTransaction and getExecutionTransaction function
* minor changes
* Allow execute if threshold is 1 for the first tx
- Related to issue #563
- `lastTx` is required due to #489
* - Normalizing logic between createTransaction and processTransaction
- Moving shared function to a new file
* Refactor `doesTxNeedApproval` back to the `isExecution`-related meaning
* Rename function and variable names
* Add tests for `getNewTxNonce` and `shouldExecuteTransaction` functions
* Pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
* Update Tests
- remove mocked `getGnosisSafeInstanceAt`
- pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (Feature) Add checks into CI (#594)
* Repare new dev and staging enviroments to deploy (#618)
* Repare new dev and staging enviroments to deploy
- Add to `deploy_pull_requests.sh` to the new path.
- Replace upload-dir to deploy staging to the new path.
* add /app/ public path for the webapp, remove duplicate yarn build from travis
* add basename to app
* deploy pr to /app
* update travis
* linter error fixes
* fix travis build script
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
* (fix) sidebar contents' scrolling (#622)
* (add) linter sorting rules (#614)
* (add) linter sorting rules
* (fix) linting errors according to the new rules
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: nicolas <nicosampler@users.noreply.github.com>
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
* (feature): [Safe Apps] App Transaction List Details (#601)
* Adding IconText component
* Get origin for TXs in Redux
* Adding Icon and name app to transaction list
* Fix customTx condition
* Fix HTML element for text
* simplify condition
* Bug: Accounts/Network are not updated in production build (#625)
* Adding Apps info to toast notification (#621)
* Adding APP_Name for APPs TXs in toast messages
* refactor: save appId instead of appName in origin field
* adding fallback icon and message to TX apps table
* review changes
* review fixes
* force build
* update Apps list info (#629)
* update Apps list info
* fix
* Feature: replace web3connect with onboardjs (#456)
* init onboard.js
* dep bump
* initial replacement of web3connect with onboard.js
* use averta font in the modal
* update onboard.js, add handlers for session murder
* update preferred wallets
* implement reconnecting to injected provider
* fix duplicate wallet disconnected notification
* update onboard dep
* onboardjs fixes
* test onboard js with hw
* add https to rpcUrl
* Added saved wallet and transactions validation
* made Transactions validation
* bnc-onboard version updated
* Fix lock/unlock problem
* bump onboardjs version
* fixed material-ui lab version
* Update onboard version
* Added hw wallets integration
* Updated preferred wallets
* Add ledger modal
* Merge with dev
* Update onboard
* BUmp new version of onboardjs
* Added some logs
* Added some logs
* Update onboardjs version fix ledger connection
* Update onboardjs version fix ledger connection
* Bump new onboardjs version
* Update new version
* Made improvements
* yarn.lock regenration
* remove https=true
* Remove comments and added some improvements
* Updated package.json dep
* Removed unused deps
* Remove web3connect dep
Co-authored-by: lukasschor <lukas.schor@gnosis.pm>
Co-authored-by: Mati Dastugue <matiasdastugue@gmail.com>
* Passing ethBalance to Apps component (#636)
* (Fix) Update safe modal verbiage (#633)
* Fixs verbiage in update safe modal
* Fixs yarn lock
* Bump dependencies (#639)
* Use hash history (#638)
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* use hash history
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* - Updates how the transactions are ordered, now orders by nonce and then by date (#640)
* (Fix) Lowercased safes address in URL (#631)
* Return checksummed addrress from `safeParamAddressFromState` selector
* Update `yarn.lock`
* Remove redundant `toChecksumAddress` call
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
* Rebase master into development (#647)
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* V1.7.5 release (#641)
* use hash history
* update package json
* BUG: App index.html cached on production (#642)
* remove caching for webapp index.html
* keep --delete flag
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
* Bug: #482 - Address book Custom transactions Recipient validation (#577)
* (fix) text input bottom border
* Fix#482 input
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# src/components/forms/TextField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/AddressBookInput/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/TokenSelectField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/index.jsx
# yarn.lock
* Fix custom tx addresses filtering
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# yarn.lock
* Remove console logs
Fixed prettier issues
* Remove unnecessary template string
* Fix `tokenAddress` string conversion
* Use `secondaryBackground` value
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (feature) Adding origin for Apps Transactions (#576)
* Adding origin field when creates a TX
* refactor: replace list of arg by object in getApprovalTransaction and getExecutionTransaction function
* minor changes
* Allow execute if threshold is 1 for the first tx
- Related to issue #563
- `lastTx` is required due to #489
* - Normalizing logic between createTransaction and processTransaction
- Moving shared function to a new file
* Refactor `doesTxNeedApproval` back to the `isExecution`-related meaning
* Rename function and variable names
* Add tests for `getNewTxNonce` and `shouldExecuteTransaction` functions
* Pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
* Update Tests
- remove mocked `getGnosisSafeInstanceAt`
- pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (Feature) Add checks into CI (#594)
* Repare new dev and staging enviroments to deploy (#618)
* Repare new dev and staging enviroments to deploy
- Add to `deploy_pull_requests.sh` to the new path.
- Replace upload-dir to deploy staging to the new path.
* add /app/ public path for the webapp, remove duplicate yarn build from travis
* add basename to app
* deploy pr to /app
* update travis
* linter error fixes
* fix travis build script
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
* (fix) sidebar contents' scrolling (#622)
* (add) linter sorting rules (#614)
* (add) linter sorting rules
* (fix) linting errors according to the new rules
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (feature): [Safe Apps] App Transaction List Details (#601)
* Adding IconText component
* Get origin for TXs in Redux
* Adding Icon and name app to transaction list
* Fix customTx condition
* Fix HTML element for text
* simplify condition
* Bug: Accounts/Network are not updated in production build (#625)
* Adding Apps info to toast notification (#621)
* Adding APP_Name for APPs TXs in toast messages
* refactor: save appId instead of appName in origin field
* adding fallback icon and message to TX apps table
* review changes
* review fixes
* force build
* update Apps list info (#629)
* update Apps list info
* fix
* Feature: replace web3connect with onboardjs (#456)
* init onboard.js
* dep bump
* initial replacement of web3connect with onboard.js
* use averta font in the modal
* update onboard.js, add handlers for session murder
* update preferred wallets
* implement reconnecting to injected provider
* fix duplicate wallet disconnected notification
* update onboard dep
* onboardjs fixes
* test onboard js with hw
* add https to rpcUrl
* Added saved wallet and transactions validation
* made Transactions validation
* bnc-onboard version updated
* Fix lock/unlock problem
* bump onboardjs version
* fixed material-ui lab version
* Update onboard version
* Added hw wallets integration
* Updated preferred wallets
* Add ledger modal
* Merge with dev
* Update onboard
* BUmp new version of onboardjs
* Added some logs
* Added some logs
* Update onboardjs version fix ledger connection
* Update onboardjs version fix ledger connection
* Bump new onboardjs version
* Update new version
* Made improvements
* yarn.lock regenration
* remove https=true
* Remove comments and added some improvements
* Updated package.json dep
* Removed unused deps
* Remove web3connect dep
Co-authored-by: lukasschor <lukas.schor@gnosis.pm>
Co-authored-by: Mati Dastugue <matiasdastugue@gmail.com>
* Passing ethBalance to Apps component (#636)
* (Fix) Update safe modal verbiage (#633)
* Fixs verbiage in update safe modal
* Fixs yarn lock
* Bump dependencies (#639)
* Use hash history (#638)
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* use hash history
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* - Updates how the transactions are ordered, now orders by nonce and then by date (#640)
* (Fix) Lowercased safes address in URL (#631)
* Return checksummed addrress from `safeParamAddressFromState` selector
* Update `yarn.lock`
* Remove redundant `toChecksumAddress` call
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: nicolas <nicosampler@users.noreply.github.com>
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
Co-authored-by: lukasschor <lukas.schor@gnosis.pm>
Co-authored-by: Mati Dastugue <matiasdastugue@gmail.com>
Co-authored-by: francovenica <francovenica@gmail.com>
* Update safe relay and transaction endpoints
* Master into development (#655)
* Fix NODE_ENV development definition
- Define develop build for development branches.
- Use POSIX standard to call shell scripts in travis file.
* Mark travis shell scripts as executable
* Fix conditional statement at travis build.sh file
* Set development endpoints to stable staging backends
* Sync master dev (#656)
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* V1.7.5 release (#641)
* use hash history
* update package json
* BUG: App index.html cached on production (#642)
* remove caching for webapp index.html
* keep --delete flag
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: nicolas <nicosampler@users.noreply.github.com>
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
Co-authored-by: lukasschor <lukas.schor@gnosis.pm>
Co-authored-by: Mati Dastugue <matiasdastugue@gmail.com>
Co-authored-by: francovenica <francovenica@gmail.com>
Co-authored-by: Richard Meissner <rmeissner@users.noreply.github.com>
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* V1.7.5 release (#641)
* use hash history
* update package json
* BUG: App index.html cached on production (#642)
* remove caching for webapp index.html
* keep --delete flag
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* V1.7.5 release (#641)
* use hash history
* update package json
* BUG: App index.html cached on production (#642)
* remove caching for webapp index.html
* keep --delete flag
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
* Bug: #482 - Address book Custom transactions Recipient validation (#577)
* (fix) text input bottom border
* Fix#482 input
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# src/components/forms/TextField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/AddressBookInput/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/TokenSelectField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/index.jsx
# yarn.lock
* Fix custom tx addresses filtering
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# yarn.lock
* Remove console logs
Fixed prettier issues
* Remove unnecessary template string
* Fix `tokenAddress` string conversion
* Use `secondaryBackground` value
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (feature) Adding origin for Apps Transactions (#576)
* Adding origin field when creates a TX
* refactor: replace list of arg by object in getApprovalTransaction and getExecutionTransaction function
* minor changes
* Allow execute if threshold is 1 for the first tx
- Related to issue #563
- `lastTx` is required due to #489
* - Normalizing logic between createTransaction and processTransaction
- Moving shared function to a new file
* Refactor `doesTxNeedApproval` back to the `isExecution`-related meaning
* Rename function and variable names
* Add tests for `getNewTxNonce` and `shouldExecuteTransaction` functions
* Pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
* Update Tests
- remove mocked `getGnosisSafeInstanceAt`
- pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (Feature) Add checks into CI (#594)
* Repare new dev and staging enviroments to deploy (#618)
* Repare new dev and staging enviroments to deploy
- Add to `deploy_pull_requests.sh` to the new path.
- Replace upload-dir to deploy staging to the new path.
* add /app/ public path for the webapp, remove duplicate yarn build from travis
* add basename to app
* deploy pr to /app
* update travis
* linter error fixes
* fix travis build script
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
* (fix) sidebar contents' scrolling (#622)
* (add) linter sorting rules (#614)
* (add) linter sorting rules
* (fix) linting errors according to the new rules
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (feature): [Safe Apps] App Transaction List Details (#601)
* Adding IconText component
* Get origin for TXs in Redux
* Adding Icon and name app to transaction list
* Fix customTx condition
* Fix HTML element for text
* simplify condition
* Bug: Accounts/Network are not updated in production build (#625)
* Adding Apps info to toast notification (#621)
* Adding APP_Name for APPs TXs in toast messages
* refactor: save appId instead of appName in origin field
* adding fallback icon and message to TX apps table
* review changes
* review fixes
* force build
* update Apps list info (#629)
* update Apps list info
* fix
* Feature: replace web3connect with onboardjs (#456)
* init onboard.js
* dep bump
* initial replacement of web3connect with onboard.js
* use averta font in the modal
* update onboard.js, add handlers for session murder
* update preferred wallets
* implement reconnecting to injected provider
* fix duplicate wallet disconnected notification
* update onboard dep
* onboardjs fixes
* test onboard js with hw
* add https to rpcUrl
* Added saved wallet and transactions validation
* made Transactions validation
* bnc-onboard version updated
* Fix lock/unlock problem
* bump onboardjs version
* fixed material-ui lab version
* Update onboard version
* Added hw wallets integration
* Updated preferred wallets
* Add ledger modal
* Merge with dev
* Update onboard
* BUmp new version of onboardjs
* Added some logs
* Added some logs
* Update onboardjs version fix ledger connection
* Update onboardjs version fix ledger connection
* Bump new onboardjs version
* Update new version
* Made improvements
* yarn.lock regenration
* remove https=true
* Remove comments and added some improvements
* Updated package.json dep
* Removed unused deps
* Remove web3connect dep
Co-authored-by: lukasschor <lukas.schor@gnosis.pm>
Co-authored-by: Mati Dastugue <matiasdastugue@gmail.com>
* Passing ethBalance to Apps component (#636)
* (Fix) Update safe modal verbiage (#633)
* Fixs verbiage in update safe modal
* Fixs yarn lock
* Bump dependencies (#639)
* Use hash history (#638)
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* use hash history
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* - Updates how the transactions are ordered, now orders by nonce and then by date (#640)
* (Fix) Lowercased safes address in URL (#631)
* Return checksummed addrress from `safeParamAddressFromState` selector
* Update `yarn.lock`
* Remove redundant `toChecksumAddress` call
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: nicolas <nicosampler@users.noreply.github.com>
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
Co-authored-by: lukasschor <lukas.schor@gnosis.pm>
Co-authored-by: Mati Dastugue <matiasdastugue@gmail.com>
Co-authored-by: francovenica <francovenica@gmail.com>
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* use hash history
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* init onboard.js
* dep bump
* initial replacement of web3connect with onboard.js
* use averta font in the modal
* update onboard.js, add handlers for session murder
* update preferred wallets
* implement reconnecting to injected provider
* fix duplicate wallet disconnected notification
* update onboard dep
* onboardjs fixes
* test onboard js with hw
* add https to rpcUrl
* Added saved wallet and transactions validation
* made Transactions validation
* bnc-onboard version updated
* Fix lock/unlock problem
* bump onboardjs version
* fixed material-ui lab version
* Update onboard version
* Added hw wallets integration
* Updated preferred wallets
* Add ledger modal
* Merge with dev
* Update onboard
* BUmp new version of onboardjs
* Added some logs
* Added some logs
* Update onboardjs version fix ledger connection
* Update onboardjs version fix ledger connection
* Bump new onboardjs version
* Update new version
* Made improvements
* yarn.lock regenration
* remove https=true
* Remove comments and added some improvements
* Updated package.json dep
* Removed unused deps
* Remove web3connect dep
Co-authored-by: lukasschor <lukas.schor@gnosis.pm>
Co-authored-by: Mati Dastugue <matiasdastugue@gmail.com>
* Adding APP_Name for APPs TXs in toast messages
* refactor: save appId instead of appName in origin field
* adding fallback icon and message to TX apps table
* review changes
* review fixes
* force build
* Adding IconText component
* Get origin for TXs in Redux
* Adding Icon and name app to transaction list
* Fix customTx condition
* Fix HTML element for text
* simplify condition
* v1.7.3 (#611)
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Update package.json (#612)
* Add public url to prod webpack config (#615)
* add public url to prod webpack config
* modify travis/webpack config to add public url
* add slash
* cherry pick new travis yml merge commit (#620)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
* Bug: #482 - Address book Custom transactions Recipient validation (#577)
* (fix) text input bottom border
* Fix#482 input
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# src/components/forms/TextField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/AddressBookInput/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/TokenSelectField/index.jsx
# src/routes/safe/components/Balances/SendModal/screens/SendFunds/index.jsx
# yarn.lock
* Fix custom tx addresses filtering
* Merge branch 'development' of https://github.com/gnosis/safe-react into fix/#482-address-book
# Conflicts:
# yarn.lock
* Remove console logs
Fixed prettier issues
* Remove unnecessary template string
* Fix `tokenAddress` string conversion
* Use `secondaryBackground` value
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (feature) Adding origin for Apps Transactions (#576)
* Adding origin field when creates a TX
* refactor: replace list of arg by object in getApprovalTransaction and getExecutionTransaction function
* minor changes
* Allow execute if threshold is 1 for the first tx
- Related to issue #563
- `lastTx` is required due to #489
* - Normalizing logic between createTransaction and processTransaction
- Moving shared function to a new file
* Refactor `doesTxNeedApproval` back to the `isExecution`-related meaning
* Rename function and variable names
* Add tests for `getNewTxNonce` and `shouldExecuteTransaction` functions
* Pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
* Update Tests
- remove mocked `getGnosisSafeInstanceAt`
- pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
Co-authored-by: Fernando <fernando.greco@gmail.com>
* (Feature) Add checks into CI (#594)
* Repare new dev and staging enviroments to deploy (#618)
* Repare new dev and staging enviroments to deploy
- Add to `deploy_pull_requests.sh` to the new path.
- Replace upload-dir to deploy staging to the new path.
* add /app/ public path for the webapp, remove duplicate yarn build from travis
* add basename to app
* deploy pr to /app
* update travis
* linter error fixes
* fix travis build script
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
* (fix) sidebar contents' scrolling (#622)
* (add) linter sorting rules (#614)
* (add) linter sorting rules
* (fix) linting errors according to the new rules
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
Co-authored-by: Gabriel Rodríguez Alsina <gabitoesmiapodo@users.noreply.github.com>
Co-authored-by: nicolas <nicosampler@users.noreply.github.com>
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
* Repare new dev and staging enviroments to deploy
- Add to `deploy_pull_requests.sh` to the new path.
- Replace upload-dir to deploy staging to the new path.
* add /app/ public path for the webapp, remove duplicate yarn build from travis
* add basename to app
* deploy pr to /app
* update travis
* linter error fixes
* fix travis build script
Co-authored-by: David Albela Pérez <3659067+davidalbela@users.noreply.github.com>
* Adding origin field when creates a TX
* refactor: replace list of arg by object in getApprovalTransaction and getExecutionTransaction function
* minor changes
* Allow execute if threshold is 1 for the first tx
- Related to issue #563
- `lastTx` is required due to #489
* - Normalizing logic between createTransaction and processTransaction
- Moving shared function to a new file
* Refactor `doesTxNeedApproval` back to the `isExecution`-related meaning
* Rename function and variable names
* Add tests for `getNewTxNonce` and `shouldExecuteTransaction` functions
* Pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
* Update Tests
- remove mocked `getGnosisSafeInstanceAt`
- pass `safeInstance` instead of `safeAddress` to `getNewTxNonce`
Co-authored-by: Fernando <fernando.greco@gmail.com>
* Fix#597: USD value not load (#609)
* Converts all the addresses to checksum values
* Fix for empty address
* fix the order of transactions, change updateSafe to upgradeSafe to avoid naming confusion with the updateSafe action (#610)
* Fix#596: Notification when safe is already updated (#599)
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Fix notification of update if the safe is already updated
* Makes the notification clickable
Displays the notification for owners only
* Identify upgrade tx
* Add red badge to Settings tab
* Fixs Padding
Removes the red dot if the user is not an owner
Co-authored-by: Fernando <fernando.greco@gmail.com>
Fernando
Mati Dastugue
Germán Martínez
nicolas
Mikhail Mikheev
Richard Meissner
David Albela Pérez
Agustin Pane
francovenica
Gabriel Rodríguez Alsina