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

(Feature) [xDai] - Contract interaction abi lookup#1429

Merged
fernandomg merged 35 commits intofeature/#1353-xDai-compatibilityfrom
feature/#1428-xDai-contract-interaction-abi-lookup
Oct 6, 2020
Merged

(Feature) [xDai] - Contract interaction abi lookup#1429
fernandomg merged 35 commits intofeature/#1353-xDai-compatibilityfrom
feature/#1428-xDai-contract-interaction-abi-lookup

Conversation

@Agupane
Copy link
Contributor

@Agupane Agupane commented Oct 5, 2020

Closes #1428
Depends on #1422

fernandomg and others added 30 commits September 30, 2020 16:33
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
…xDai-use-generic-config

# Conflicts:
#	src/config/networks/__tests__/networks.test.ts
#	src/config/networks/network.d.ts
…ion file

Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
- also added 'local' as a network for testing purposes
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
…ai-use-generic-config

# Conflicts:
#	src/config/networks/__tests__/networks.test.ts
#	src/config/networks/mainnet.ts
#	src/config/networks/network.d.ts
#	src/config/networks/rinkeby.ts
#	src/config/networks/xdai.ts
#	src/logic/wallets/getWeb3.ts
Moves getContractABI to src/config/index.ts
@Agupane Agupane requested a review from fernandomg October 5, 2020 14:35
@Agupane Agupane self-assigned this Oct 5, 2020
@github-actions
Copy link

github-actions bot commented Oct 5, 2020

CLA Assistant Lite All Contributors have signed the CLA.

@Agupane Agupane changed the base branch from development to feature/#1415-xDai-use-generic-config October 5, 2020 14:36
@github-actions
Copy link

github-actions bot commented Oct 5, 2020

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 0 0
Ignored 1 N/A
  • Result: ✅ success
  • Annotations: 0 total

Report generated by eslint-plus-action

@ghost
Copy link

ghost commented Oct 5, 2020

Travis automatic deployment:
https://pr1429--safereact.review.gnosisdev.com/app

@Agupane Agupane changed the base branch from feature/#1415-xDai-use-generic-config to feature/#1353-xDai-compatibility October 5, 2020 15:27
…ai-contract-interaction-abi-lookup

# Conflicts:
#	src/config/__tests__/config.test.ts
#	src/config/index.ts
#	src/logic/contractInteraction/sources/EtherscanService.ts
#	src/routes/safe/components/Balances/SendModal/screens/ContractInteraction/utils/index.ts
…ai-contract-interaction-abi-lookup

# Conflicts:
#	src/routes/safe/components/Balances/SendModal/screens/ContractInteraction/utils/index.ts
@ghost
Copy link

ghost commented Oct 5, 2020

Travis automatic deployment:
https://pr1429--safereact.review.gnosisdev.com/app

@ghost
Copy link

ghost commented Oct 6, 2020

Travis automatic deployment:
https://pr1429--safereact.review.gnosisdev.com/app

@Agupane Agupane marked this pull request as ready for review October 6, 2020 12:57
Copy link
Contributor

@fernandomg fernandomg left a comment

Choose a reason for hiding this comment

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

Tested and worked fine. Left a suggestion, but not a showstopper.

@francovenica francovenica removed their request for review October 6, 2020 16:09
@fernandomg fernandomg merged commit 5278527 into feature/#1353-xDai-compatibility Oct 6, 2020
@fernandomg fernandomg deleted the feature/#1428-xDai-contract-interaction-abi-lookup branch October 6, 2020 16:14
@github-actions github-actions bot locked and limited conversation to collaborators Oct 6, 2020
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.

4 participants