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 comments on Address.js #477

Open
HarisSpahijaPon opened this issue Feb 14, 2020 · 1 comment
Open

Remove comments on Address.js #477

HarisSpahijaPon opened this issue Feb 14, 2020 · 1 comment
Labels
good first issue For issues that a new contributor could likely submit a pull request for without needing much help help wanted For issues that have a clear solution described and are not currently prioritized core team work severity low

Comments

@HarisSpahijaPon
Copy link
Contributor

HarisSpahijaPon commented Feb 14, 2020

Type: minor

On line 84 on Address.js

        break;
      // case "region":
      //   addressElement = <Fragment>{addressProp} </Fragment>;
      //   break;
      default:
        addressElement = (

Commented case should be removed

@aldeed aldeed added good first issue For issues that a new contributor could likely submit a pull request for without needing much help help wanted For issues that have a clear solution described and are not currently prioritized core team work severity low labels Feb 26, 2020
@skodamarthi
Copy link

@aldeed @HarisSpahijaPon Can I work on this as part of hacktoberfest?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue For issues that a new contributor could likely submit a pull request for without needing much help help wanted For issues that have a clear solution described and are not currently prioritized core team work severity low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants