Skip to content

Convert CustomizeGasModal and GasModalCard to ES6 classes#7776

Closed
whymarrh wants to merge 2 commits intoMetaMask:developfrom
whymarrh:es6-classes-customize-gas-modal
Closed

Convert CustomizeGasModal and GasModalCard to ES6 classes#7776
whymarrh wants to merge 2 commits intoMetaMask:developfrom
whymarrh:es6-classes-customize-gas-modal

Conversation

@whymarrh
Copy link
Copy Markdown
Contributor

@whymarrh whymarrh commented Jan 9, 2020

Refs #7766

This PR converts the CustomizeGasModal and GasModalCard components to ES6 classes.


if (gasPriceChanged) {
this.setState({
UNSAFE_componentWillReceiveProps (nextProps, _) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why add the second parameter?

@Gudahtt
Copy link
Copy Markdown
Member

Gudahtt commented Jan 10, 2020

Oh, it looks like this component isn't used anymore.

@whymarrh whymarrh deleted the es6-classes-customize-gas-modal branch January 10, 2020 19:06
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.

2 participants