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

Modal window missing version number #55

Closed
johnalarcon opened this issue Mar 29, 2021 · 0 comments
Closed

Modal window missing version number #55

johnalarcon opened this issue Mar 29, 2021 · 0 comments
Assignees
Labels
Status: Accepted This item has been accepted and will be implemented or otherwise addressed. Type: Defect This item is an error or other behavior that negatively impacts code execution.

Comments

@johnalarcon
Copy link
Collaborator

Description

The modal popup window is missing the "Compatible up to:" version number. The update client file is requesting the current version via an option when the value is actually stored as a transient. This issue doesn't hinder operation of the Update Manager plugin or the update client file, however, it is important for UX. Because this issue lives in the update client file, it will have to be a major version release as that would be a breaking change.

The problematic code:

$this->cp_latest_version = get_option('cp_latest_version', '');

Solution

Store and retrieve version string from the correct context by replacing the above-noted code with the following. This solution is not yet tested and will need further study before implementation.

$this->cp_latest_version = get_transient('codepotent_update_manager_cp_version');
if (empty($this->cp_latest_version)) {
	$this->cp_latest_version = $this->get_latest_version_number();
}

image

@johnalarcon johnalarcon self-assigned this Mar 29, 2021
@johnalarcon johnalarcon added Type: Defect This item is an error or other behavior that negatively impacts code execution. Status: Accepted This item has been accepted and will be implemented or otherwise addressed. labels May 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Accepted This item has been accepted and will be implemented or otherwise addressed. Type: Defect This item is an error or other behavior that negatively impacts code execution.
Projects
None yet
Development

No branches or pull requests

1 participant