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

Custom labels #630

Closed
wants to merge 28 commits into from
Closed

Custom labels #630

wants to merge 28 commits into from

Conversation

AquiGorka
Copy link
Contributor

@AquiGorka AquiGorka commented Feb 25, 2019

These changes:

Videos (updated March 4th):

Mobile:
https://www.useloom.com/share/2cbf05eea7624abeb306a9cd86a406f2

Desktop:
https://www.useloom.com/share/a3bc58e6f76448b8ba21309986ea0fb0

  • Updates React to use hooks
  • Preferences prototype
  • Updates to open/close Preferences
  • Adds CustomLabelModal prototype
  • Mock Custom Labels manager (will remove once integration with aragon.js is done)
  • Add custom label action button to IdentityBadge component (aragon/ui#327)
  • Set label
  • Remove all labels
  • Import .json
  • Export .json
  • Styling
  • Make it work for iframe apps
  • Update all IdentiyBadge usage to allow custom labels
  • AnimaterPreferences modal show/hide

@AquiGorka AquiGorka force-pushed the feature/custom-labels branch 2 times, most recently from a89a660 to 03389a9 Compare February 26, 2019 08:33
@AquiGorka AquiGorka force-pushed the feature/custom-labels branch 2 times, most recently from 8b91bea to 0d10020 Compare February 27, 2019 10:47
@sohkai sohkai added this to the A1 Sprint: 4.2 milestone Feb 27, 2019
@sohkai sohkai mentioned this pull request Feb 27, 2019
2 tasks
@AquiGorka AquiGorka force-pushed the feature/custom-labels branch 5 times, most recently from 4e34d37 to 708301b Compare February 28, 2019 16:27
@AquiGorka AquiGorka force-pushed the feature/custom-labels branch 2 times, most recently from fe98d26 to 5870cc9 Compare March 4, 2019 07:54
@sohkai sohkai modified the milestones: A1 Sprint: 4.2, A1 Sprint: 4.3 Mar 4, 2019
@luisivan
Copy link
Contributor

luisivan commented Mar 4, 2019

@AquiGorka I left only @sohkai and @bpierre as reviewers since they are the only ones required to review this PR

@AquiGorka AquiGorka force-pushed the feature/custom-labels branch 4 times, most recently from 7bee92c to 42c41bc Compare March 4, 2019 11:28
@AquiGorka AquiGorka marked this pull request as ready for review March 6, 2019 12:29
@@ -37,10 +37,11 @@
"lodash.uniqby": "^4.7.0",
"onecolor": "^3.0.5",
"prop-types": "^15.6.2",
"react": "^16.7.0",
"react": "^16.8.3",
"react-blockies": "^1.3.0",
"react-container-dimensions": "^1.3.3",
"react-dom": "^16.7.0",
Copy link
Contributor

@2color 2color Mar 7, 2019

Choose a reason for hiding this comment

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

I think react-dom also needs to be upgraded to to 16.8.x to utilise hook support. I got this error: https://reactjs.org/warnings/invalid-hook-call-warning.html.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Shoot! You are correct. I'll wait till we merge your PR so no need to update here.

@luisivan luisivan removed this from the A1 Sprint: 4.3 milestone Mar 18, 2019
@AquiGorka
Copy link
Contributor Author

Closing in favor of #656

@AquiGorka AquiGorka closed this Mar 28, 2019
@AquiGorka AquiGorka deleted the feature/custom-labels branch March 28, 2019 08:49
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.

6 participants