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

Edit no longer available in the free version #1151

Closed
AndreeaCristinaRadacina opened this issue May 15, 2024 · 8 comments
Closed

Edit no longer available in the free version #1151

AndreeaCristinaRadacina opened this issue May 15, 2024 · 8 comments
Assignees
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. Priority-Medium Expected resolution time - up to 1 month. small (1-3h) This label is used for issues that can be completed within 3 hours or less.

Comments

@AndreeaCristinaRadacina

Description

Users reported that they can no longer edit their charts after the update.

ref: https://secure.helpscout.net/conversation/2597422525/418824?viewId=931544

Step-by-step reproduction instructions

  1. Select the Visualizer in the left hand menu
  2. Click the gear icon under that graph to edit/update.

Screenshots, screen recording, code snippet or Help Scout ticket

image

Environment info

No response

Is the issue you are reporting a regression

Yes, this is a regression.

@AndreeaCristinaRadacina AndreeaCristinaRadacina added the bug This label could be used to identify issues that are caused by a defect in the product. label May 15, 2024
@pirate-bot pirate-bot added customer report Indicates the request came from a customer. regression Issue represents a change in behavior from a previous version that is not intended or desired.. labels May 15, 2024
@AndreeaCristinaRadacina
Copy link
Author

@vytisbulkevicius
Copy link
Contributor

I can't replicate this issue, @AndreeaCristinaRadacina were you able to replicate it?

Even if I create bar chart with the old version of the plugin, I update plugin then I can still edit the free chart.

@GrigoreMihai since you worked on this, do you know what could be the issue?

@vytisbulkevicius vytisbulkevicius added the Priority-Medium Expected resolution time - up to 1 month. label May 15, 2024
@GrigoreMihai GrigoreMihai added the small (1-3h) This label is used for issues that can be completed within 3 hours or less. label May 16, 2024
@GrigoreMihai
Copy link
Contributor

@vytisbulkevicius I can not replicate the issue either. Most likely he is using a pro chart type, but there is no reference of the chart he is using in the ticket.

From the background of the provided image I think that he is using the column chart which is pro: https://vertis.d.pr/i/duA3cB.

@AndreeaCristinaRadacina My suggestion would be to get more info from the user regarding the type of chart (and some print screens) he is using to be able to pinpoint an exact issue if there is one.

@AndreeaCristinaRadacina
Copy link
Author

@vytisbulkevicius @GrigoreMihai I wasn't able to replicate it either, but after seeing multiple tickets on the same topic, I thought it would be better to share this with you.

Regarding the chart type, I am not sure if this should be the behavior, as we didn't mention anything about charts being removed from the free version for the existing users.

I asked the user for the information, I will keep you updated.

@AndreeaCristinaRadacina
Copy link
Author

@GrigoreMihai, the user from wp.org mentioned he is using bar charts -
image

Please let me know if there's something else I should do to help.

@selul
Copy link
Contributor

selul commented May 16, 2024

fmho it's normal to get multiple users reporting this since we did not had this before.

@vytisbulkevicius
Copy link
Contributor

Yes, I think there can be cases where some users had pro charts installed somehow (maybe used pro version before) but if it's really a Bar chart as one user mentioned it shouldn't happen as Bar chart was free and we didn't change it. I see there are 2 users now that reached out about this even though "bar" chart was mentioned it looks like they had column chart in use from that image in the background and if that's true it's expected.

@vytisbulkevicius vytisbulkevicius removed the regression Issue represents a change in behavior from a previous version that is not intended or desired.. label May 16, 2024
@AndreeaCristinaRadacina
Copy link
Author

I clarified this with both users. They were using pro charts.

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. customer report Indicates the request came from a customer. Priority-Medium Expected resolution time - up to 1 month. small (1-3h) This label is used for issues that can be completed within 3 hours or less.
Projects
None yet
Development

No branches or pull requests

5 participants