Skip to content
This repository was archived by the owner on Nov 10, 2023. It is now read-only.

Conversation

@dasanra
Copy link
Collaborator

@dasanra dasanra commented Mar 10, 2021

Added

Fixed

Chore

matextrem and others added 27 commits February 23, 2021 19:25
Mergeback release v3.1.0 to development
* Types in safeDataExtractor.ts

* Adds useEstimateSafeCreationGas hook

* Refactor Layout.tsx

* Remove gas price in estimateGasForDeployingSafe

* Types in createSafe and adds gasLimit to deployment transaction

* Uses useEstimateSafeCreationGas in ReviewComponent for transaction creation

* Add types

* Fix types

* Fix types of ReviewInformation modal on safe load

* Fix safe creation for multiple owners

* fix unit tests

* Remove unnecesary minimum tx gas

Co-authored-by: Mati Dastugue <[email protected]>
Co-authored-by: nicolas <[email protected]>
Co-authored-by: nicosampler <[email protected]>
Co-authored-by: Daniel Sanchez <[email protected]>
Mergeback CI fixes to development
* Enable Wallet connect on EWC
* set owners and threshold properly

* Take threshold from step 2
* Cleanup old transaction list files

* Rename view folder from GatewayTransactions to TxList

* Remove all reducer logic for incomingTransactions

* Delete allTransactionsLogic

* Cleanup cancel transaction legacy logic

* Remove fetch creation transaction

* Remove moduleTransactions logic

* Remove unused transfer methods

* Cleanup unused types
* Replace collectibles fetch with client gateway

* Replaces balance endpoint with client gateway

* Remove default export of tokens list

* Set the default rows per page to 100

* Fix ether price load

* Remove Add custom token button

* Remove add custom asset and add custom token modals
* add external icon in safe version (settings)

* change wording in receive modal

* fix Modal titles and color on back/cancel button

* move toggle to the left
(contract interaction modal)

* add title "Method" for drop down

* review lettercase in titles and buttons

* add contract address label in contract interaction

* fix label for input "Contract address"

* fix label when use custom data (Contract address)

* fix external link component on Safe version

* change Label "Recipient" to "Contract address"
* Upgrade dependencies without breaking changes

* Remove travis leftovers

* Upgrade types dependencies

* Upgrade eslint rules to new prettier configuration
* Rename custom transaction to custom interaction

* fix no transaction img size
* remove url as required attribute of manifest.json for safe-apps

* getAppInfoFromUrl: get requiredData from manifest
@dasanra dasanra self-assigned this Mar 10, 2021
@github-actions
Copy link

CLA Assistant Lite All Contributors have signed the CLA.

@github-actions
Copy link

github-actions bot commented Mar 10, 2021

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type Occurrences Fixable
Errors 0 0
Warnings 14 0
Ignored 10 N/A
  • Result: ✅ success

  • Annotations: 14 total


[warning] @typescript-eslint/explicit-module-boundary-types

Require explicit return and argument types on exported functions' and classes' public class methods


Report generated by eslint-plus-action

@github-actions
Copy link

@github-actions
Copy link

@dasanra dasanra requested a review from francovenica March 10, 2021 10:56
@github-actions
Copy link

@francovenica
Copy link
Contributor

Can't confirm these 2:

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

1 similar comment
@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@francovenica
Copy link
Contributor

francovenica commented Mar 11, 2021

Issues NFT's:

The NFT's are still showing in the list after a transfer, even when they already are showing in the recipient safe where they were sent
It has been over 20 mins since the tx was executed at the moment of writing this

Safe sent from: https://pr2015--safereact.review.gnosisdev.com/rinkeby/app/#/safes/0xfB9d4e27aA63f51DeE7c644e82d49D60AC975838/balances

Safe that received it:
https://pr2015--safereact.review.gnosisdev.com/rinkeby/app/#/safes/0x9913B9180C20C6b0F21B6480c84422F6ebc4B808/balances

The tx:
https://rinkeby.etherscan.io/tx/0xc7e318fdf58cf249ebe4133b4567546fccd9a094119a47249d23ddb84475baff

image.png
image.png

Question:
We made that safes 1.0.0 can see NFT's? I don't remember that being a thing.
This safe 1.0.0 has one
https://pr2015--safereact.review.gnosisdev.com/rinkeby/app/#/safes/0xeca13481F8514752dC08106c7A265c85caFa563E/balances/collectibles

@francovenica
Copy link
Contributor

Update: The token is not longer showing in the safe where it was transferred from. Is weird that it took more than 30 minutes to disappear

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

Copy link
Contributor

@francovenica francovenica left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good enough for Staging

@dasanra dasanra merged commit 99f4604 into master Mar 12, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Mar 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants