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

Add back Google Table charts #524

Closed
contactashish13 opened this issue Sep 6, 2019 · 17 comments
Closed

Add back Google Table charts #524

contactashish13 opened this issue Sep 6, 2019 · 17 comments
Labels
enhancement Request to improve or optimize an existing feature or functionality in the project
Milestone

Comments

@contactashish13
Copy link
Contributor

contactashish13 commented Sep 6, 2019

https://app.asana.com/0/1129400186358085/1136935251791377/f

@contactashish13 contactashish13 added the enhancement Request to improve or optimize an existing feature or functionality in the project label Sep 6, 2019
@DanyCaissy DanyCaissy added this to the 3.4.4 milestone Mar 5, 2020
@contactashish13
Copy link
Contributor Author

contactashish13 commented Apr 30, 2020

PR #686 and https://github.com/Codeinwp/visualizer-pro/pull/264 are ready to test.

I would suggest the following steps to test:

  • use an old version of the plugin that supported Google table charts (I think before 3.0.12) and create a few table charts
  • upgrade to the latest available version and create a few DataTable table charts
  • when you upgrade to this PR all table charts created in the previous steps should be visible/searchable/editable
  • all table charts should be insertable/editable in the block editor
  • new table charts can be created using either Google or DataTable

PR https://github.com/Codeinwp/visualizer-pro/pull/264 will succeed once free is released.

This also resolves https://github.com/Codeinwp/visualizer-pro/issues/256

Please don't merge the PRs as I need to add cypress tests

@contactashish13
Copy link
Contributor Author

@rodica-andronache
Copy link
Contributor

@contactashish13 the only issue I saw is with editing old Google Table charts.

How to replicate it:

  • Install & activated https://downloads.wordpress.org/plugin/visualizer.3.1.0.zip
  • Create a few table charts
  • Update the plugin to the latest version
  • Update the plugin to this PR
  • Manual edit the Google Table charts created before, with the 3.1.0 version (no matter the editor, Simple, Text or Excel, both in the Gutenberg editor or in the Visualizer library)

What happens is that after save, all the values in the table gets an Array before.
download (1)

Regarding https://github.com/Codeinwp/visualizer-pro/issues/256 I've changed the Precision option to -1 for one of the charts, and I got the same error mentioned in the issue, not sure if I'm missing anything
Screenshot (6)

@contactashish13
Copy link
Contributor Author

@rodica-andronache v3.1.0 did not have google table charts - it had DataTable table charts. Also, google table charts was part of Pro so can you please mention the combination of free and pro you have used?

I believe you typed a precision of -1 instead of selecting it using the number selector (which will not allow you to go below 0). I have handled that case as well.

@rodica-andronache
Copy link
Contributor

@contactashish13 you were right, I wasn't using the right version.
I've tested again the migration with Visualizer 3.0.12 and Visualizer PRO 1.7.8.

So, what I did is:

  1. Install & activated Visualizer 3.0.12 and Visualizer PRO 1.7.8.
  2. Create a few table charts
  3. Update both plugins to their latest versions
  4. Update the plugins to the new PRs

The only 2 issues I've seen are:

  1. The old Table Google charts ( created with the old versions ) does not show up when filtering by type or library type.
    Here are all the charts
    download (1)
    Filtering Google chart tables does not show those charts
    download (2)

  2. If I use the Visualizer block to display any of the charts, and Manul Edit the chart, and click on the Save button, the button doesn't seem to work. It stays in a loading state forever.
    download (3)

You can see this here
https://simple-moth.jurassic.ninja/
User: demo
Password: QIfhpw0Aex92oIG3pnXOxvDZ

@contactashish13
Copy link
Contributor Author

@rodica-andronache When the old table charts are upgraded to the new version, an option gets inserted in the options (wp_options) table with a key of visualizer-upgraded, which shows the version of the plugin that you had active before upgrading to this new version. I don't see that in the database (I installed phpmyadmin plugin):

image

Maybe this is related to the second problem I fixed below.

You may want to try this again on a new instance (or by using the WP Reset plugin to clear the database). Once you upgrade to this PR, you should see 2 options visualizer-upgraded and visualizer-upgraded-from.

The second problem should be okay now. It had something to do with the error you see even on the login page:

image

@rodica-andronache
Copy link
Contributor

@contactashish13 the second issue was fixed. The first one still persists for me.
I've tried both on my local and on a new Jurassic instance here
https://certain-hoary.jurassic.ninja/wp-admin/
User: demo
pass: Ok4PTb8DPu$Qi1Md%)P@zt7c
In both cases, filtering charts does not take into account the old Table Google charts.
Here are the two options in both cases:
on jurasssic: https://prnt.sc/syfmbh
on my local: https://prnt.sc/syfrxi

@contactashish13
Copy link
Contributor Author

@rodica-andronache I'm not sure I understand the problem. You can see that the charts have had their types changed and they are visible in the library.

image

@rodica-andronache
Copy link
Contributor

@contactashish13
Here are all the charts
download (13)

Filtering Table GoogleCharts does not show the charts created with the old version
download (14)

@contactashish13
Copy link
Contributor Author

@rodica-andronache The screenshots helped! I've fixed this.

@rodica-andronache
Copy link
Contributor

@contactashish13 that's working well now 👍
Saw another issue which I think is related to these PRs ( tested with the current versions and the issue is not there ).
Similar to #688 when creating a DataTable Table chart, insert it in a post using the Visualizer block, and clicking on Advanced Options, the block breaks.
This happens just for DataTable Table charts, not Google Table charts.
Screenshot 2020-06-17 at 12 49 10

@contactashish13
Copy link
Contributor Author

@rodica-andronache have fixed this.

And also found another problem: in googlecharts formatting of date/number columns does not work in the block editor. I believe it is an issue even with the currently released version.

@rodica-andronache
Copy link
Contributor

@contactashish13 I've tested again and everything seems right for me. If you can also fix the conflicts here https://github.com/Codeinwp/visualizer-pro/pull/264 I think we can merge these, if there aren't other things that you think needs to be done.

I didn't get exactly what's the new issue you've mentioned. If there's not something related to this maybe you can add it on git, with more details?
Thanks

@contactashish13
Copy link
Contributor Author

@rodica-andronache I've fixed the conflicts.

The issue I have found is this. Can you confirm this behavior at your end too? I'd prefer such issues to be created by you as you are able to provide more details than I can.

  1. Create any type of core google chart (line. bar, column etc)
  2. Change formatting, which reflects on the chart. Save the chart.
    image
  3. Insert that chart in a block
  4. Change formatting. It does not change anything. These fields don't seem to work for any chart.
    image

@rodica-andronache
Copy link
Contributor

@contactashish13 is happening for me too, on the current version as well. I've added it here #724

@rodica-andronache
Copy link
Contributor

@contactashish13 saw one more thing that I've missed before. In the Help tab, the tables are called now Tabular instead of Table, making the Video Demo link to point to a resource that doesn't exist.
download (17)

How it is in the current released version
download (18)

@contactashish13
Copy link
Contributor Author

PR #725 should take care of this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Request to improve or optimize an existing feature or functionality in the project
Projects
None yet
Development

No branches or pull requests

3 participants