-
Notifications
You must be signed in to change notification settings - Fork 110
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.
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
Fix missing exported changes when batching operations #1294
Fix missing exported changes when batching operations #1294
Conversation
Revert angular demo to previous version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this contribution. Good job!
Just a few formalities before we can merge the pull request:
- please add a changelog entry
- please add a unit test that demonstrates the bug (it should fail before applying your fix). File
computation-suspension.spec.ts
seems to be the best place for it.
@@ -8,7 +8,7 @@ more details. | |||
## Demo | |||
|
|||
<iframe | |||
src="https://codesandbox.io/embed/github/handsontable/hyperformula-demos/tree/2.5.x/angular-demo?autoresize=1 | |||
src="https://codesandbox.io/embed/github/handsontable/hyperformula-demos/tree/2.4.x/angular-demo?autoresize=1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please pull the latest changes from develop
Codecov Report
@@ Coverage Diff @@
## develop #1294 +/- ##
========================================
Coverage 97.20% 97.20%
========================================
Files 167 167
Lines 14295 14295
Branches 3064 3064
========================================
Hits 13895 13895
Misses 395 395
Partials 5 5
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, thank you!
Context
Fixes an issue where the wrong address key was being used (col, row) instead of (row, col).
72862d7#diff-85d401cfd01ec0afae371f4af4e54e3a13cc6bcefe1e2d411671c013acc8a8f6R67-R70
How did you test your changes?
Added two tests, one in
content-changes.spec.ts
and another incomputation-suspension.spec.ts
.Types of changes
Related issues:
Checklist: