Skip to content

Add NETWORK_TYPE_RPC constant#10203

Merged
rekmarks merged 3 commits intodevelopfrom
network-display-props-refactor
Jan 20, 2021
Merged

Add NETWORK_TYPE_RPC constant#10203
rekmarks merged 3 commits intodevelopfrom
network-display-props-refactor

Conversation

@rekmarks
Copy link
Copy Markdown
Member

@rekmarks rekmarks commented Jan 16, 2021

Pending #10200

  • Adds a NETWORK_TYPE_RPC constant for the 'rpc' network type and uses it wherever appropriate.
  • Refactors the NetworkDisplay component's props, which could be significantly simplified.
  • Removes network constants from `ui/app/helpers/constants/common' and replaces their usage with shared constants

@rekmarks rekmarks changed the title Refactor NetworkDisplay props Add NETWORK_TYPE_RPC constant Jan 16, 2021
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [0c806e6]
Page Load Metrics (517 ± 64 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint358552126
domContentLoaded33793451513364
load33993551713364
domInteractive33793451513364

@brad-decker
Copy link
Copy Markdown
Contributor

LGTM, the only remaining instances of 'rpc' as a standalone string are in migrations.

Base automatically changed from rationalize-constants to develop January 20, 2021 16:13
@rekmarks rekmarks dismissed a stale review January 20, 2021 16:13

The base branch was changed.

@rekmarks rekmarks requested a review from brad-decker January 20, 2021 16:14
@rekmarks rekmarks force-pushed the network-display-props-refactor branch from 0c806e6 to 3f03425 Compare January 20, 2021 16:17
@rekmarks rekmarks marked this pull request as ready for review January 20, 2021 16:17
@rekmarks rekmarks requested a review from a team as a code owner January 20, 2021 16:17
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [3f03425]
Page Load Metrics (804 ± 82 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaintNaNNaNNaNNaNNaN
domContentLoaded410103080316981
load411103680417082
domInteractive409103080216981

@rekmarks rekmarks merged commit 7077ee6 into develop Jan 20, 2021
@rekmarks rekmarks deleted the network-display-props-refactor branch January 20, 2021 16:49
@github-actions github-actions bot locked and limited conversation to collaborators Jan 20, 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.

3 participants