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

fix(frontend): solve is define or is assigned but never use issues #837

Merged
merged 2 commits into from
Jun 22, 2021
Merged

fix(frontend): solve is define or is assigned but never use issues #837

merged 2 commits into from
Jun 22, 2021

Conversation

leisterfrancisco
Copy link
Member

@leisterfrancisco leisterfrancisco commented Jun 22, 2021

Resolves #838

What does this PR do?

Solve is defined or is assigned but never use issues

How should this be tested?

  • Make run
  • Check there is no console log errors with defined or assigned status

@leisterfrancisco leisterfrancisco added the dont-merge DO NOT MERGE YET label Jun 22, 2021
@@ -522,7 +522,6 @@ const EditProfileBankMobile = ({ profile, onSubmit, setField, loading, userName
InputLabelProps={{
shrink: true
}}
className={classes.textField}
Copy link
Member Author

Choose a reason for hiding this comment

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

@Edgar097 could you please check if this no cause any problem, the thing is that there was two className declaration to same component.

@leisterfrancisco leisterfrancisco removed the dont-merge DO NOT MERGE YET label Jun 22, 2021
@leisterfrancisco leisterfrancisco self-assigned this Jun 22, 2021
@Edgar097 Edgar097 merged commit b84d8e1 into eoscostarica:master Jun 22, 2021
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.

ES LINT: Remove unused variables
3 participants