* 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 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>
* Fix material-ui/lab version
* Fix breaking changes introduced in commit b960dbf3cf36774a878239c9fed800b58d08abcd
* Fix web3connect version to `1.0.0-beta.25`
* components v2
* Fix type
* Adding TextBox component and some minor changes
* replacing harcoded color by the one defined in varibales
* TextBox: Replacing div by p
Works fine for me:
- The address book selector displays
- Typing the name/address will filter
- Not case sensitive
- Deleted won't show anymore. New added addresses will show and filter
- Works for regular Tx and Custom Tx
This is out of the scope of this issue, but worth mentioning: the issue reported in #482 is still here: the custom Tx still show addresses that are not valid (non contract addresses).
Works as intent ended, the address is automatically filled when you add a new address from a Tx.
Checked the Address book functionality in general making sure nothing broke by this fix.
David Albela Pérez
Fernando
Agustin Pane
Mikhail Mikheev
francovenica
nicolas
Gabriel Rodríguez Alsina
Miguel Mota