Generator-jhipster: [React] Use hooks for components

Created on 11 Jun 2019  路  16Comments  路  Source: jhipster/generator-jhipster

Use hooks in all React components

Hooks are available since the version 16.8 of React. The goal will be to integrate hooks in JHipster.

Todo

Here are all the components that seem be impacted :

  • [x] Header #9883 MERGED
  • [x] Password #9899 MERGED
  • [x] Register #9899 MERGED
  • [x] Configuration #9919 MERGED
  • [x] Health #9921 MERGED
  • [x] Logs #9921 MERGED
  • [x] Activate #9964 MERGED
  • [x] Logout #9968 MERGED
  • [x] Settings #9965 MERGED
  • [x] Metrics #9966 MERGED
  • [x] PasswordResetFinish #9924 MERGED
  • [x] UserManagementDetail #9929 MERGED
  • [x] UserManagementDeleteDialog #9938 MERGED
  • [x] UserManagementUpdate #9938 MERGED
  • [x] Home #9969 MERGED
  • [x] Login #9971 MERGED
  • [x] App #9972 MERGED
  • [x] Audits #9975 MERGED
  • [x] UserManagement #9981 MERGED

And the entities :

  • [x] Entity #9983
$$ bug-bounty $$ $500 area work in progress react

Most helpful comment

@jdubois I have changed the bounty amount as this is a lot of work and deserves more. Hope you are fine with it

All 16 comments

@qmonmert I suggest doing small PRs so that its easy to review and test, may a PR per module(admin, layout, account, entity)

@deepu105 yes I'm doing some small PRs

Once we implement the new bounty system, this issue should get 300$ bounty atleast

@jdubois I have changed the bounty amount as this is a lot of work and deserves more. Hope you are fine with it

Of course I am!!

Crazy work here @qmonmert !
Hope someone with more experience with React can review all your PRs

thanks @pascalgrimaud :) I hope cc @deepu105 & @chrisdns ;)

@deepu105 & @pascalgrimaud I am not able to make azure e2e tests pass for PRs #9968 & #9981 even after multiple attempts (it works locally). Are these PR still mergeable or should we wait for success ?

excellent work @qmonmert :clap:

Yes nice work @qmonmert !
About the 2 others PRs, sorry I was busy the last weeks... I'll have a look when possible

@qmonmert : I'm increasing the bounty, as it was really too much work, and you're still working on it to achieve this enhancement. So it will be well deserved

Already paid, I trust you :-P

thanks @jdubois :)

you're too fast @jdubois !
thanks for the hard work @qmonmert !!

thanks @pascalgrimaud ;)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

frantzynicolas picture frantzynicolas  路  3Comments

Steven-Garcia picture Steven-Garcia  路  3Comments

ahmedeldeeb25 picture ahmedeldeeb25  路  3Comments

edvjacek picture edvjacek  路  3Comments

DanielFran picture DanielFran  路  3Comments