Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Fix incorrect admin notice text when auto-updates are enabled for a plugin when JS is not enabled in the browser #130

Merged
merged 4 commits into from
May 16, 2020

Conversation

pbiron
Copy link
Contributor

@pbiron pbiron commented May 14, 2020

This partially addresses #129.

Creating this as a draft PR, pending a solution to the other aspect of #129 (i.e., the Enable/Disable links not being bound to Ajax actions in the search results).

…lugin when JS is not enabled in the browser.

No equivalent problem exists for disabling plugin auto-updates nor on either the multisite or non-multisite themes screens.
@pbiron pbiron added the bug Something isn't working label May 14, 2020
@pbiron pbiron added this to the 0.8.1 milestone May 14, 2020
@pbiron pbiron self-assigned this May 14, 2020
FIxes it to work after replacing the list-table on plugin search.
Chatting with @pbiron, this looks better.
@pbiron pbiron marked this pull request as ready for review May 16, 2020 23:46
@pbiron
Copy link
Contributor Author

pbiron commented May 16, 2020

thanx @azaozz that does allow the Enable/Disable links to work after a search. Going to merge this.

@pbiron pbiron merged commit 36ebad4 into master May 16, 2020
@azaozz azaozz deleted the fix/non-ajax-plugin-enable-admin-notice-incorrect branch May 17, 2020 02:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants