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

Enable/Disable wallets by configuration flag#1504

Merged
dasanra merged 44 commits intodevelopmentfrom
feature/disableWallet
Oct 23, 2020
Merged

Enable/Disable wallets by configuration flag#1504
dasanra merged 44 commits intodevelopmentfrom
feature/disableWallet

Conversation

@matextrem
Copy link
Contributor

Closes #1481

josipbagaric and others added 30 commits September 29, 2020 11:23
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Mati Dastugue <mdastugu@amazon.com>
Co-authored-by: Matias Dastugue <matias.dastugue@altoros.com>
Co-authored-by: Mati Dastugue <mdastugu@amazon.com>
* Filter apps by network

* add missing network
Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
Co-authored-by: fernandomg <fernando.greco@gmail.com>
Co-authored-by: Agustin Pane <agustin.pane@gmail.com>
* Add staging xDai deployment to travis

* Add safe apps production url for xdai

* Add travis rule to build xDai on master or integration branch
* fix min-height app containter and margin to info text at
bottom

* change css unit from % to vh

Co-authored-by: nicosampler <nf.dominguez.87@gmail.com>
* Replace all harcoded values

* fix `generateBatchRequest` calls

* Fix some ETH harcoded values + Add tests

* Fix null or undefinded value

* Move nativeCoin outside component + fix toTokenInput function

* Remove native coin from useEffect

Co-authored-by: fernandomg <fernando.greco@gmail.com>
Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
)

* Adds FEATURES enum
Renames features to disabledFeatures

* Uses FEATURES enum instead of hardcoded contract features

* Refactor enabledFeatures function, now checks that there are not disabled features by config before return the enabledFeatures result

* Adds SAFE_APPS and CONTRACT_INTERACTION to FEATURES enum

* Remove SAFE_APPS from sidebar if disabled

* Redirect user from /apps if safe apps are disabled

* Disables contract interaction from config

* Remove !remoteOwners

* Move settings to a constant
fernandomg and others added 6 commits October 15, 2020 14:19
* add equality function to prevent re-renders

* fix typing

- I left the inline equalityFn, as an ad-hoc solution.
First, because we're going to remove this when #1479 is done
Second, because this way it infers the types defined in the useSelector generics
Co-authored-by: nicosampler <nf.dominguez.87@gmail.com>
@matextrem matextrem added Major Needs to be fixed for immediate next public release. Feature 👑 New functionality labels Oct 20, 2020
@matextrem matextrem self-assigned this Oct 20, 2020
@github-actions
Copy link

CLA Assistant Lite All Contributors have signed the CLA.

@github-actions
Copy link

github-actions bot commented Oct 20, 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 3 N/A
  • Result: ✅ success
  • Annotations: 0 total

Report generated by eslint-plus-action

@ghost
Copy link

ghost commented Oct 20, 2020

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

@dasanra dasanra requested a review from francovenica October 21, 2020 11:55
@ghost
Copy link

ghost commented Oct 21, 2020

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

@ghost
Copy link

ghost commented Oct 21, 2020

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

@ghost
Copy link

ghost commented Oct 22, 2020

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

@francovenica
Copy link
Contributor

Checked in the PR first which is in the rinkeby network. All wallets are available
Switched to my local env configured with the xDai network. The Ledger and Trezor direct connections were disabled (no longer show in the onboardjs modal)

Looks good to me

@ghost
Copy link

ghost commented Oct 23, 2020

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

@dasanra dasanra merged commit c9e9fe8 into development Oct 23, 2020
@dasanra dasanra deleted the feature/disableWallet branch October 23, 2020 11:30
@github-actions github-actions bot locked and limited conversation to collaborators Oct 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Feature 👑 New functionality Major Needs to be fixed for immediate next public release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[xDai] Feature flag implementation for supported wallets

9 participants