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

Visualizer 3.9.2 Import CSV data from URL broken #970

Closed
digser opened this issue Dec 23, 2022 · 5 comments · Fixed by #972
Closed

Visualizer 3.9.2 Import CSV data from URL broken #970

digser opened this issue Dec 23, 2022 · 5 comments · Fixed by #972
Labels
bug This label could be used to identify issues that are caused by a defect in the product. released Indicate that an issue has been resolved and released in a particular version of the product.

Comments

@digser
Copy link

digser commented Dec 23, 2022

Description:

When trying to import data in CSV format into a Visualizer table either from a file or URL the whirring wheel is shown into perpetuity.
Rolling back to 3.9.1 fixes the fault. I have rolled versions forward and back three times with the same results.

How to reproduce:

1.Create a new table
2.Try to import from a CSV using the tabular.csv example
3.

Expected behaviour:CSV data loads

Current behaviour: data does not load

Reference:

Technical info

  • WordPress version: 6.1.1
  • Plugin version: 3.9.2
@girishpanchal30
Copy link
Contributor

I've reviewed this and I can't replicate this issue. Can you please share more details?

Screencast: https://www.loom.com/share/a3319d03960241709c756a4709f79090

Thanks

@MarinescuLucia
Copy link

@girishpanchal30 This occurs when you go back to edit a chart, after it was previously created. It doesn't happen the first time when you create the chart. I'm able to replicate the issue - screencast.

Reported here.

@girishpanchal30 girishpanchal30 added the bug This label could be used to identify issues that are caused by a defect in the product. label Dec 29, 2022
@girishpanchal30
Copy link
Contributor

@MarinescuLucia,

Thanks for the explanation! Now the issue is fixed in this PR #971, please check with the build zip

Thanks

@MarinescuLucia
Copy link

@girishpanchal30 Tested the PR and the chart is displayed properly now. Thanks!

@vytisbulkevicius vytisbulkevicius linked a pull request Jan 5, 2023 that will close this issue
vytisbulkevicius added a commit that referenced this issue Jan 5, 2023
- Fixed the chart refresh problem when importing from CSV #970
pirate-bot pushed a commit that referenced this issue Jan 5, 2023
##### [Version 3.9.3](v3.9.2...v3.9.3) (2023-01-05)

- Fixed the chart refresh problem when importing from CSV [#970](#970)
@pirate-bot
Copy link
Contributor

🎉 This issue has been resolved in version 3.9.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This label could be used to identify issues that are caused by a defect in the product. released Indicate that an issue has been resolved and released in a particular version of the product.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants