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

prevent unexpected behaviour which generated by orphaned license keys #4669

Closed
3 tasks done
Benunc opened this issue Apr 23, 2020 · 1 comment · Fixed by #4696
Closed
3 tasks done

prevent unexpected behaviour which generated by orphaned license keys #4669

Benunc opened this issue Apr 23, 2020 · 1 comment · Fixed by #4696
Assignees

Comments

@Benunc
Copy link
Member

Benunc commented Apr 23, 2020

Bug Report

User Story

As a user, I only want accurate license key errors so that the WP dashboard/admin interface is not crowded with incorrect messages.

Details

Any user with old license keys saved in the database (a number that will grow over time as more people purchase plans to replace single add-ons and legacy bundles) gets a phantom warning related to expired/inactive license keys when in reality their plan license keys are active.

Current Behavior

Currently, users with old license expired license keys have no way of removing them (and thus the warnings on every admin screen) unless they know the workaround in my video below.

This creates extra work for the user, as well as a burden on the Support and Customer Success teams as they have to have access to the site to resolve the problem for the customer.

Expected Behavior

All licenses saved in the database should display on the licenses screen and be able to be deactivated or deleted completely.

Bug Type

  • I am not sure whether this functionality ever worked as expected.

Steps to Reproduce

The steps involve adding a live license key and then expiring it, which needs someone with EDD access to the site to facilitate. Here's a video: https://screencast-o-matic.com/watch/cYf0oAA0JT

Here's what I did before that video:

  1. Add license for a single add-on to the site.
  2. Log in as EDD-level admin on the givewp.com site, and manually switch that license to expired.
  3. Refresh the licenses on the licenses tab of the test site.

Related

We regularly have calls about this.

Acceptance Criteria

  • All licenses in the DB show on the back end.
  • The error messages related to licenses only show if there are no active plan licenses.
@ravinderk ravinderk self-assigned this Apr 29, 2020
@ravinderk
Copy link
Collaborator

Discussion: https://givewp.slack.com/archives/C0E290T6Z/p1588178782102500

We discussed this issue and shortlisted the following issues with license functionality.

  1. "Unauthorized access" notice when update addon.
  2. Single addon expired licenses notices confuse admin who upgraded to bundle plan.
  3. A rare admin behavior also detected on the license setting page where the admin is able to see a single license key separately even it is part of the bundle plan.

@ravinderk ravinderk changed the title fix: prevent orphaned license keys from creating phantom alerts prevent unexpected behaviour which generated by orphaned license keys May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants