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

Decimal Numbers are ignored and not displayed #955

Closed
Skystream96 opened this issue Oct 23, 2022 · 1 comment · Fixed by #956 or #958
Closed

Decimal Numbers are ignored and not displayed #955

Skystream96 opened this issue Oct 23, 2022 · 1 comment · Fixed by #956 or #958
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

@Skystream96
Copy link

Description:

If you try to create a Table Chart with one of the columns containing data that has decimals, you'll notice that when showing the chart, the said column doesn't have any data. Also, if you try to manual edit the data, you'll notice that it dissapeared.

How to reproduce:

  1. Have Visualizer installed and a set of data with decimals
  2. Create a new Table Chart and Import the CSV
  3. Oberve how the decimal data is not displayed
  4. Try to manual edit the data and notice how the data is not there

Expected behaviour:

Decimals are interpreted and shown in the chart

Current behaviour:

Decimal data is ignored

Reference:

User reported here:
Screencast showing the issue

Data Example Here
workplace-safety-score.csv

Testing Instance:https://critiqueinsect.s3-tastewp.com/wp-admin
User: matei@themeisle.com
Password: H4rFBq%OackaNwuOA0FM2o4S

Technical info

  • WordPress version: 6.0.3
  • Plugin version: 3.8.0
@Skystream96 Skystream96 added the bug This label could be used to identify issues that are caused by a defect in the product. label Oct 23, 2022
girishpanchal30 added a commit that referenced this issue Oct 27, 2022
@girishpanchal30 girishpanchal30 linked a pull request Oct 27, 2022 that will close this issue
@vytisbulkevicius vytisbulkevicius linked a pull request Nov 4, 2022 that will close this issue
vytisbulkevicius added a commit that referenced this issue Nov 10, 2022
- Fixed table chart decimal number issue Decimal Numbers are ignored and not displayed #955
- Fix the scrollbar issue that happens on WP 6.1 version [WP 6.1] Chart permissions selector is not scrollable #950
- Fix PHP Warning Warning when copy (frontend action) is enabled #949
- Compatibility with the WPML translation plugin for translating charts [PRO feature] add support for string translations #731
- Integration with Woocommerce Data endpoints for creating charts [PRO feature]
- Show new features on chart library page
pirate-bot pushed a commit that referenced this issue Nov 10, 2022
#### [Version 3.9.0](v3.8.1...v3.9.0) (2022-11-10)

- Fixed table chart decimal number issue Decimal Numbers are ignored and not displayed [#955](#955)
- Fix the scrollbar issue that happens on WP 6.1 version [WP 6.1] Chart permissions selector is not scrollable [#950](#950)
- Fix PHP Warning Warning when copy (frontend action) is enabled [#949](#949)
- Compatibility with the WPML translation plugin for translating charts [PRO feature] add support for string translations [#731](#731)
- Integration with Woocommerce Data endpoints for creating charts [PRO feature]
- Show new features on chart library page
@pirate-bot
Copy link
Contributor

🎉 This issue has been resolved in version 3.9.0 🎉

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 Nov 10, 2022
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.

2 participants