Skip to content

bug(Changing data in rows passed into useTable does not update rows): Fix bug causing updated data not updating rows due to improper memoization#16

Merged
Buuntu merged 1 commit intomasterfrom
gabud/fix-memoize-bug
Aug 21, 2020
Merged

bug(Changing data in rows passed into useTable does not update rows): Fix bug causing updated data not updating rows due to improper memoization#16
Buuntu merged 1 commit intomasterfrom
gabud/fix-memoize-bug

Conversation

@Buuntu
Copy link
Copy Markdown
Owner

@Buuntu Buuntu commented Aug 21, 2020

  • Fixes bug causing updated data not updating rows due to improper memoization

Fix bug causing updated data not updating rows due to improper memoization
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #16 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   99.44%   99.46%   +0.01%     
==========================================
  Files           4        4              
  Lines         181      186       +5     
  Branches       38       40       +2     
==========================================
+ Hits          180      185       +5     
  Misses          1        1              
Impacted Files Coverage Δ
src/hooks.tsx 99.32% <100.00%> (+0.02%) ⬆️

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 b448904...98ae578. Read the comment docs.

@Buuntu Buuntu merged commit 687bd1f into master Aug 21, 2020
@Buuntu Buuntu deleted the gabud/fix-memoize-bug branch August 21, 2020 20:18
Buuntu added a commit that referenced this pull request Aug 21, 2020
…#16)

Fix bug causing updated data not updating rows due to improper memoization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants