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.

Sign up for GitHub

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

Jump to bottom

Divide DataTable & upgrade react-dom #81

Merged

tnrich merged 2 commits into master from divideDataTable

Jul 2, 2024

Contributor

jgespinosa10 commented Jun 28, 2024

Divide DataTable into smaller chunks. The file is still too big, but this pr took around 400 lines out.
Replaced react-dom render with the new version root and replaced react-dom utilities with useRef and createRef.
Took mobx out of uploader component.
refactor UploadCsvWizard so it uses hooks instead of connect

jgespinosa10 force-pushed the divideDataTable branch 2 times, most recently from 3f60429 to 18a75c6 Compare

June 28, 2024 18:15


          chore: divide data table to smaller chunks

19d9867

jgespinosa10 force-pushed the divideDataTable branch from 18a75c6 to 87c7b5c Compare

July 1, 2024 23:18


          fix: upgrade react-dom

5af38fa

jgespinosa10 force-pushed the divideDataTable branch from 87c7b5c to 5af38fa Compare

July 2, 2024 15:38

Collaborator

tnrich commented Jul 2, 2024

Thanks @jgespinosa10 looks gr8!

tnrich merged commit e53eedd into master

14 of 15 checks passed

jgespinosa10 deleted the divideDataTable branch

July 2, 2024 18:40

jgespinosa10 mentioned this pull request

Revert "Divide DataTable & upgrade react-dom" #98

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet