search

broadinstitute / seqr

web-based analysis tool for rare disease genomics
GNU Affero General Public License v3.0
176 stars 88 forks source link

React Hooks? #1428

Open NLSVTN opened 4 years ago

NLSVTN commented 4 years ago

Hi,

We are wondering why you have not yet taken advantage of the new feature 'React Hooks'?

https://reactjs.org/docs/hooks-reference.html

Is there a particular reason for not doing that for this particular project?

hanars commented 4 years ago

we do not have the bandwidth on our team to refactor, and the code works as is. If you want to help us by updating to use the latest features, we would be more than happy to accept a pull request! If not, I'll leave the ticket open as this would be best practice for us to do at some point in the future when we have time

NLSVTN commented 4 years ago

Oh, ok, I see. We may start using them then and if any of our refactored components might be of any use for you too, I will issue PR.