* fix: No threshold change for owners replacement
* fix: Extract added owner from addressBook
When replacing an owner, the added owner's name was the same as the removed one
* fix: Add or Update addressBook entry for the newly added owner
* Replace `.then` with `async/await`
* Optimize AddressBook entry `name` update
* fix: store added owner in the addressBook
* Revert "Optimize AddressBook entry `name` update"
This reverts commit 00a75d15
* refactor: AddressBook entry from plain JS object to immutable Record
* fix: merge instead of set for the AddressBookEntry record
* refactor: addOrUpdateAddressBookEntry redux action
changed signature to `addOrUpdateAddressBookEntry(entryAddress, entry)`
Where `entry` is an object with only the required fields to be updated
* refactor: update `addOrUpdateAddressBookEntry` call, due to signature change
* offchain signatures wip
* offchain signing wip
* offchain signatures wip
* offchain signatures wip
* save signatures to the history service
* fix eth signer & useEfefct hook
* offchain signatures wip
* signature check, mainnet testing wip
* dep update
* disable offchain signing for smart contract wallets
* Refactor EIP712 signer
* bring back .env.example
* Check if save version is >1.1.1
* use canTryoffchainSigning boolean variable, add comment about 4001 error
* move semver selector for safe version/offchain signatures to a constant, make use of empty_data for isContractWallet
* remove TYPE when sending txs to history service
* add eth_signTypedData_v4 signer, dep bump, add missing await
* add comments about version check for canTryOffchainSigning variable
* hide "please sign notification"
* dep bump
* dep bump
* Check if connected is ledger before trying offchain signatures
* minor fixes, temp deployment to test trezor
* add hardwareWallet boolean property to wallet model, disable offchain signatures for hw wallets
* add personal signer
* prettier fixes
* offchain signatures fixes
* fix: No threshold change for owners replacement
* fix: Extract added owner from addressBook
When replacing an owner, the added owner's name was the same as the removed one
* fix: Add or Update addressBook entry for the newly added owner
* Replace `.then` with `async/await`
* Optimize AddressBook entry `name` update
* Revert "Optimize AddressBook entry `name` update"
This reverts commit 00a75d15
* refactor: AddressBook entry from plain JS object to immutable Record
* fix: merge instead of set for the AddressBookEntry record
* refactor: addOrUpdateAddressBookEntry redux action
changed signature to `addOrUpdateAddressBookEntry(entryAddress, entry)`
Where `entry` is an object with only the required fields to be updated
* offchain signatures wip
* offchain signing wip
* offchain signatures wip
* offchain signatures wip
* save signatures to the history service
* fix eth signer & useEfefct hook
* offchain signatures wip
* signature check, mainnet testing wip
* dep update
* disable offchain signing for smart contract wallets
* Refactor EIP712 signer
* bring back .env.example
* Check if save version is >1.1.1
* use canTryoffchainSigning boolean variable, add comment about 4001 error
* move semver selector for safe version/offchain signatures to a constant, make use of empty_data for isContractWallet
* remove TYPE when sending txs to history service
* add eth_signTypedData_v4 signer, dep bump, add missing await
* add comments about version check for canTryOffchainSigning variable
* hide "please sign notification"
* dep bump
* dep bump
* Check if connected is ledger before trying offchain signatures
* minor fixes, temp deployment to test trezor
* add hardwareWallet boolean property to wallet model, disable offchain signatures for hw wallets
* Generates a cache to avoid multiples getHumanFriendlyToken() for the same token address
* Adds etags implementation for transactions
* Caches outgoing and incoming safe transactions based on etag value
* Removes cachedSafeTransactions, cachedSafeIncommingTransactions
* Refactors getTokenInstance
* Avoid recreating tokens on fetchTokens() once we have them in redux
* Fixs error on catch
* Batch request tokens balances
* Fixs missing token names
Changes the tokens limit from 300 to 3000
* fix: failed to instantiate non-standard ERC-20 tokens
For the batchRequest of balances, we're just using the `balanceOf` method call. So having a simple ABI with that only method prevents errors with non-standard ERC-20 Tokens.
* Removes unnecessary action updateSafeThreshold
Removes unnecessary action fetchEtherBalance
* Updated comments in code
Replaces constant with directly dispatching action
* BatchRequest done right
* fix: invalid action name `savedToken` -> `saveToken`
* Renames getTokenInstance to getTokenInfos
Fixs first load of transactions are empty
* Move fetchTokenBalances to `Balances` and `SendModal` components
* fix: Incoming transaction type
Backend now changed the type from 'incoming' to one of: `'ERC721_TRANSFER', 'ERC20_TRANSFER', 'ETHER_TRANSFER'`
* fix: tokenInstance `symbol` and `decimal` extraction
* Fix property name `decimals` instead of `tokenDecimals`
* Standardize non-standard ERC20 tokens discovery
* fix: isStandardERC20
* Revert "Move fetchTokenBalances to `Balances` and `SendModal` components"
This reverts commit ed84bd92
* Fixs Typo INCOMING_TX_TYPES
Renames tokenInstance with localToken
* Renames getBatchBalances to getTokenBalances
Returns saved tokens instead of tokenInstance in getTokenInfos
* Remove promise returns
Co-authored-by: fernandomg <fernando.greco@gmail.com>
* Stepper component
* proxyfactory web3 contract
* add styles to body steps
* Steps info
* Open component: moving from class to function
* remove opening route and rendering it in Open component instead
* recover safe creation from txHash in localStorage
* remove commented code
* restore commented code
* creatign TX fix
* fix createSafe then function
* fixing stepper
* remove unused code
* remove opening route and finishin both variants of create
* add loader dots svg
* add error state design and loader dots
* fix error section
* add description to steps
* adding etherscan link
* taking values from variables
* fix heigh in body content
* add success svg
* add check image on last step
* fix margin and heigt to body rows
* remove commented code
* remove commented code
* fix for #396
* Fix empty_code
* set error if getReceipt fails
* fixes
* Fix: remove txHash from pendingInfo on retry
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
Co-authored-by: Agustín Longoni <agustin.longoni@altoros.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
* 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
* Refactor Balances tabs: reduces the amount of props received, exported tokens lists to a component
* Refactor Balances tabs: reduces the amount of props received, exported tokens lists to a component
* 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
* Displays the assets in the manage list
* Fixs add custom token/asset modal cancel button
* 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
* Implements manage list to add/remove assets
* Implements manage list to add/remove assets
* Merge branch 'feature/#469-ERC721-feature-implementation' of https://github.com/gnosis/safe-react into feature/#469-ERC721-feature-implementation
# Conflicts:
# src/routes/safe/components/Balances/Collectibles/index.jsx
# src/routes/safe/components/Balances/index.jsx
# src/utils/constants.js
* Implements blacklisted assets
* 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
* Adds initial components for AddCustomAsset support
* 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`
* Fixs erc721Enabled check
* Adds margin to modal icon
Fix search bar
* Fix hidding buttons in coins table
* Fixs display all available assets by default
* Fixs modal assets
* Fixs blacklistedAssetsAddresses save to storage
Fixs show token button
Co-authored-by: fernandomg <fernando.greco@gmail.com>
Co-authored-by: Gabriel Rodriguez Alsina <gabriel.rodriguez@altoros.com>
* Fixs duplicated notifications
* Implements feedback, now the displayed txHash are stored on localstorage and once the first time we notify the user about it, they won't never appear again
* Uses the last time the user logged in
* Fix safe version null check
Fixs date string comparison
Adds the safe address to the check of last time logged in
* (Fix) Wrong value for ERC-20 tokens transfers (#679)
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
* (Fix) send tx from address book (#677)
* 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
* bump version in package.json (#683)
* Onboardjs - Get wallets by platform, fix ledger/trezor rpcUrls (#665)
* Added desktop support
* Added desktop option and logs for testing
* Get desktop mode from window object
* Add torus support and remove logs
* Update function name
* Upgrade onboardjs version
* fix ledger/trezor rpcUrls
* Bump version to 1.8.3
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
* Master/dev conflict fix (#685)
* Added desktop support
* Added desktop option and logs for testing
* Get desktop mode from window object
* Add torus support and remove logs
* Update function name
* Upgrade onboardjs version
* v1.8.2 (#682)
* (Fix) Wrong value for ERC-20 tokens transfers (#679)
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
* (Fix) send tx from address book (#677)
* 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
* bump version in package.json (#683)
Co-authored-by: Fernando <fernando.greco@gmail.com>
* fix ledger/trezor rpcUrls
* Bump version to 1.8.3
Co-authored-by: Mati Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
Co-authored-by: Mati Dastugue <matias.dastugue@altoros.com>
* Added desktop support
* Added desktop option and logs for testing
* Get desktop mode from window object
* Add torus support and remove logs
* Update function name
* Upgrade onboardjs version
* v1.8.2 (#682)
* (Fix) Wrong value for ERC-20 tokens transfers (#679)
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
* (Fix) send tx from address book (#677)
* 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
* bump version in package.json (#683)
Co-authored-by: Fernando <fernando.greco@gmail.com>
* fix ledger/trezor rpcUrls
* Bump version to 1.8.3
Co-authored-by: Mati Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Fernando <fernando.greco@gmail.com>
* Added desktop support
* Added desktop option and logs for testing
* Get desktop mode from window object
* Add torus support and remove logs
* Update function name
* Upgrade onboardjs version
* fix ledger/trezor rpcUrls
* Bump version to 1.8.3
Co-authored-by: Mikhail Mikheev <mmvsha73@gmail.com>
* (Fix) Wrong value for ERC-20 tokens transfers (#679)
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
* (Fix) send tx from address book (#677)
* 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
* bump version in package.json (#683)
Co-authored-by: Fernando <fernando.greco@gmail.com>
* 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>
Mikhail Mikheev
Fernando
nicolas
Agustin Pane
Mati Dastugue
Germán Martínez
lukasschor
Richard Meissner
David Albela Pérez