Skip to content
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

[Bug]: set formula content includes row range will throw error! #1297

Open
IMSupperkaka opened this issue Aug 14, 2023 · 4 comments
Open

[Bug]: set formula content includes row range will throw error! #1297

IMSupperkaka opened this issue Aug 14, 2023 · 4 comments
Labels
Bug Something isn't working Impact: High

Comments

@IMSupperkaka
Copy link

Description

See demo for details

Video or screenshots

No response

Demo

https://codesandbox.io/s/jolly-bell-l7693t?file=/src/index.js

HyperFormula version

2.4.0

Your framework

No response

Your environment

chrome 113

@adrianszymanski89
Copy link
Contributor

Hi @IMSupperkaka

It seems that the reason for this issue was the order of the operations. When I changed to the following order - row 0, row 0, row 1, row 2 it worked correctly. Can you please confirm if that's your expected output?

Here's the changed example: https://codesandbox.io/s/heuristic-wiles-h3dx42

@IMSupperkaka
Copy link
Author

Hi @IMSupperkaka

It seems that the reason for this issue was the order of the operations. When I changed to the following order - row 0, row 0, row 1, row 2 it worked correctly. Can you please confirm if that's your expected output?

Here's the changed example: https://codesandbox.io/s/heuristic-wiles-h3dx42

It seems a dependencyGragh bug, i fixed in my local version.
I know it works when i change the order of operation but In some reason i canot control the order of operation.

@IMSupperkaka
Copy link
Author

In fact, I found dozens of bugs in the process of development, can I submit some Pr later?

@adrianszymanski89
Copy link
Contributor

Hi @IMSupperkaka

I understand. Of course, we really appreciate our users' input, so feel free to create PR.

@sequba sequba added Bug Something isn't working Impact: High labels Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Impact: High
Projects
None yet
Development

No branches or pull requests

3 participants