-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: custom network component #7159
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #7159 +/- ##
==========================================
- Coverage 34.60% 34.59% -0.01%
==========================================
Files 1017 1017
Lines 27144 27150 +6
Branches 2205 2211 +6
==========================================
Hits 9393 9393
- Misses 17262 17268 +6
Partials 489 489
☔ View full report in Codecov by Sentry. |
...mponents/Views/Settings/NetworksSettings/NetworkSettings/CustomNetworkView/CustomNetwork.tsx
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wachunei please run the smoketests against this branch on bitrise. Let us know if you need assistance on doing that.
Bitrise pipelines started here https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/f051934b-663a-49ac-bc06-0dc0839cb9af cc @chrisleewilcox |
@chrisleewilcox should I re run smoke tests? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! LGTM!
Passing smoke tests were requested: app.bitrise.io/app/be69d4368ee7e86d/pipelines/c9f35e14-4f47-4a38-9c0a-f677162d58ca
Kudos, SonarCloud Quality Gate passed! |
Development & PR Process
release-xx
label to identify the PR slated for a upcoming release (will be used in release discussion)needs-dev-review
label when work is completedneeds-qa
: PR requires manual QA.No QA/E2E only
: PR does not require any manual QA effort. Prior to merging, ensure that you have successful end-to-end test runs in Bitrise.Spot check on release build
: PR does not require feature QA but needs non-automated verification. In the description section, provide test scenarios. Add screenshots, and or recordings of what was tested.QA Passed
label when QA has signed off (Only required if the PR was labeled withneeds-qa
)team-
(orexternal-contributor
label if your not a MetaMask employee)Description
This PR refactors the CustomNetwork component to include it in #7169 feat(ramp): add NetworkSwitcher view
Changes
Add
onNetworkSwitch
that overrides default navigation behaviorshowAddedNetworks
to show currently added network, changing the action from "Add" to "Switch" networkcustomNetworksList
that overrides the currentPopularList
variable being read inside the component"Switch"
stringChange
ImageIcon
component to aAvatarNetwork
component that will takeimageSource
or{ uri: imageUrl }
, defaulting to the circled network initial letter in case none is presenttoggleWarningModal
is not passed as propNetwork
interface to addrpcPrefs.imageUrl
as optional propertyCustomNetworkProps
interface extension ofNetworks
QA
This PR must not change current only reference of this component
metamask-mobile/app/components/Views/Settings/NetworksSettings/NetworkSettings/index.js
Lines 1092 to 1102 in 5d964f6
Screenshots/Recordings
~~ If applicable, add screenshots and/or recordings to visualize the before and after of your change~~
Issue
fixes #???
Checklist