hyperformula icon indicating copy to clipboard operation
hyperformula copied to clipboard

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

Open IMSupperkaka opened this issue 2 years ago • 4 comments

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

IMSupperkaka avatar Aug 14 '23 06:08 IMSupperkaka

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

adrianszymanski89 avatar Aug 14 '23 09:08 adrianszymanski89

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 avatar Aug 14 '23 11:08 IMSupperkaka

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

IMSupperkaka avatar Aug 14 '23 11:08 IMSupperkaka

Hi @IMSupperkaka

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

adrianszymanski89 avatar Aug 14 '23 12:08 adrianszymanski89