#37550 closed defect (bug) (fixed)
Shiny Updates: minor, edge case, visual glitch when bulk updating
Reported by: | afercia | Owned by: | ocean90 |
---|---|---|---|
Milestone: | 4.6 | Priority: | normal |
Severity: | normal | Version: | 4.6 |
Component: | Plugins | Keywords: | has-screenshots has-patch commit dev-reviewed |
Focuses: | ui, javascript | Cc: |
Description
Minor, edge case but worth considering. To reproduce:
- select at least one installed plugin to update checking its checkbox
- select "Update" from the Bulk Action select
- disconnect from the Internet :) to intentionally trigger an error
- press the "Apply" button to update
- an error notice will be displayed in the plugin row and one more notice on top of the page, let's focus on the one in the plugin row
Now, say the error was just temporary, a network hiccup or whatever and you want to retry to update so:
- reconnect to the Internet :)
- select again the plugin with the error notice still displayed in place
- select "Update" from the Bulk Action select
- press the "Apply" button to update
The error notice icon will start spinning (that's supposed to happen just for the "update available" icon) and the message "Updating..." appears with the error styling. Also, when the update completes, the success message "Updated!" will be displayed in the notice with the error styling.
Attachments (4)
Change History (15)
This ticket was mentioned in Slack in #feature-shinyupdates by ocean90. View the logs.
8 years ago
This ticket was mentioned in Slack in #feature-shinyupdates by ocean90. View the logs.
8 years ago
#6
in reply to:
↑ 4
@
8 years ago
Replying to ocean90:
37550.patch fixes the random "error" message which isn't helpful at all. It reuses the string from the editor.
Moved into #37583.
#7
@
8 years ago
- Keywords commit added
37550.2.diff is 37550.diff but without the error message part which has been moved into #37583.
This ticket was mentioned in Slack in #core by ocean90. View the logs.
8 years ago
#9
@
8 years ago
- Keywords dev-reviewed added
Reproduced the issue and tested 37550.2.diff. Looks good.
37550.patch fixes the random "error" message which isn't helpful at all. It reuses the string from the editor.