Skip to content
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

Remove undeclared name variable from render #8336

Merged
merged 1 commit into from
Apr 15, 2020

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Apr 14, 2020

The header component for the new Signature Request screen has an undeclared variable called name in it. This was present in the original implementation of this component in #6891. It's unclear what this was supposed to be, and it doesn't seem to reference anything that exists.

The header component for the new Signature Request screen has an
undeclared variable called `name` in it. This was present in the
original implementation of this component in #6891. It's unclear what
this was supposed to be, and it doesn't seem to reference anything that
exists.
@Gudahtt Gudahtt force-pushed the remove-non-existent-prop branch from e33fcaf to a382129 Compare April 14, 2020 20:53
@metamaskbot
Copy link
Collaborator

Builds ready [a382129]
Page Load Metrics (589 ± 52 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeNotificationfirstPaint34494042
domContentLoaded34971558711053
load35171658910952
domInteractive34971558711053

Copy link
Contributor

@whymarrh whymarrh left a comment

Choose a reason for hiding this comment

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

LGTM!

@Gudahtt Gudahtt merged commit ec40c1f into develop Apr 15, 2020
@Gudahtt Gudahtt deleted the remove-non-existent-prop branch April 15, 2020 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants