Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Respecting standards for tokenreg dapp #2287

Merged
merged 5 commits into from
Sep 23, 2016
Merged

Respecting standards for tokenreg dapp #2287

merged 5 commits into from
Sep 23, 2016

Conversation

ngotchac
Copy link
Contributor

  • Added Copyrights to JS and CSS files
  • Use as few [...].bind(this) as possible
  • Replaced most let by const
  • Added isRequired to required PropTypes

@parity-cla-bot
Copy link

It looks like this contributor signed our Contributor License Agreement. 👍

Many thanks,

Ethcore CLA Bot

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M5-ui labels Sep 23, 2016
@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 23, 2016
@jacogr jacogr merged commit 5c6d6b4 into js Sep 23, 2016
@jacogr jacogr deleted the ng-tokenreg-standard branch September 23, 2016 18:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ngotchac @parity-cla-bot @jacogr