search

specify / specify7

Specify 7
https://www.specifysoftware.org/products/specify-7/
GNU General Public License v2.0
66 stars 36 forks source link

Rewrite Business rules, resourceApi.js and collectionApi.js to TypeScript #2621

Open maxpatiiuk opened 1 year ago

maxpatiiuk commented 1 year ago

... and make them follow closer the React philosophy

This will add TypeScript coverage to the most crucial files of Specify

As part of the rewrite, should also extend the unit tests and integration tests.

Also, get rid of the event triggers in favor of creating new objects and letting it propagate downward like in react components

maxpatiiuk commented 1 year ago

FYI, fixing this and https://github.com/specify/specify7/issues/2622 would allow to get rid of jQuery, Backbone and Underscore.js dependencies

maxpatiiuk commented 1 year ago

I begun writing down few features to consider adding/chaning when doing the rewrite: https://github.com/specify/specify7/blob/185c27b283bee3decabdb84970ca6a0b06a9ef78/specifyweb/frontend/js_src/lib/hooks/store.tsx#L17-L32

I likely won't have time to finish this (or likely to even start this), so I will make sure to expand on this with much more detail on what I see as a good design for the refactored version