[#24893] - [4.0] Update check alerts
- Fixed in Code Base
- 14 May 2019
- Medium
- Build: 4.0-dev
- # 24893
- Diff
- brianteeman:alert
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
In J3 the quickicon for the new version etc where hidden in the bottom left of the control panel so we had a popup alert as well. As the quickicons are now bigger and right at the top we don't need the popup as well. Also you will see that the popups over wrote each other so they were even more useless
This PR removes the popup and associated strings and ensures that the icons get the red danger instead of the brown warning and also standardises the 3 strings so that the count is in the same place.
Testing Instructions
Will require npm i
To test extension updates - install a language and then change the version number in the db
To test joomla updates - create a custom extension server
Before
After
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration Language & Strings JavaScript Repository NPM Change Front End Plugins |
| Labels |
Added:
?
NPM Resource Changed
?
|
||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-05-14 13:21:55 |
| Closed_By | ⇒ | wilsonge |
Looks good to me. Yeah I'm not sure exactly why that is - i guess something is theoretically wrong somewhere there :/ but as long as it works :/
Thanks
Note:
Taking off the Messages is not the solution imho. We have an underlying issue.
The same situation happens when one updates many items at the same time: the message only displays for the last item updated.
Here for languages:
Install older packs
de-DE_joomla_lang_full_3.9.5v1.zip
fr-FR_joomla_lang_full_3.9.5v2.zip
This is what you get when you update:
What I meant is that the ideal for these checks would be to combine the Messages in a unique Warning message with as many items concerned as necessary.
That was a side effect bonus of the issue that was being solved here. Please create your own issue for the messages
@infograf768 there's definitely a bigger problem to solve in speed of messages being removed. However in this specific case we had both alerts and the warning buttons at the top of the screen. Which meant we had two alerts for the same thing.
@wilsonge I updated jupdatecheck.es6.js as suggested
I didnt see anything in extensionupdatecheck.es6.js to update ??