-
Notifications
You must be signed in to change notification settings - Fork 29
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
Comments
PR #686 and https://github.com/Codeinwp/visualizer-pro/pull/264 are ready to test. I would suggest the following steps to test:
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 |
@rodica-andronache this resolves https://github.com/Codeinwp/visualizer-pro/issues/256 as well. |
@contactashish13 the only issue I saw is with How to replicate it:
What happens is that after save, all the values in the table gets an Regarding https://github.com/Codeinwp/visualizer-pro/issues/256 I've changed the |
@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 |
@contactashish13 you were right, I wasn't using the right version. So, what I did is:
The only 2 issues I've seen are:
You can see this here |
@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 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 The second problem should be okay now. It had something to do with the error you see even on the login page: |
@contactashish13 the second issue was fixed. The first one still persists for me. |
@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. |
@contactashish13 Filtering Table GoogleCharts does not show the charts created with the old version |
@rodica-andronache The screenshots helped! I've fixed this. |
@contactashish13 that's working well now 👍 |
@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. |
@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? |
@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. |
@contactashish13 is happening for me too, on the current version as well. I've added it here #724 |
@contactashish13 saw one more thing that I've missed before. In the |
PR #725 should take care of this. |
https://app.asana.com/0/1129400186358085/1136935251791377/f
The text was updated successfully, but these errors were encountered: