change hooks order execution when sorting

[lalogrosz]

When you sort and you are in some page, the sort action reset the currentPage to 1 always.
Right now I don't want to add some logic to keep the currentPage, but I prefer to know first the sorting and then the changePage event, otherwise I don't know if the changePage is triggered by the pagination or by the sort.

Changing the useDidUpdateEffect order resolves this.

[codecov]

Codecov Report

Merging #812 (6faf952) into next (61b1d3f) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             next     #812   +/-   ##
=======================================
  Coverage   26.08%   26.08%           
=======================================
  Files          42       42           
  Lines         897      897           
  Branches      328      328           
=======================================
  Hits          234      234           
  Misses        659      659           
  Partials        4        4           

Impacted Files	Coverage Δ
src/DataTable/DataTable.tsx	0.00% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61b1d3f...6faf952. Read the comment docs.

[jbetancur]

Thanks for the PR. Do you see any cases where this might break other functionality? Do you have an example of what this is resolving for you?

[lalogrosz]

No, I didn't see any other scenario.