User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Updated language files
Updated quicklink plug to add new conditions, add a quicklinks group and restore the warning notice
use a 3.9 site and set the updater to custom url. Add the github url and see that the update works and that the plugin is updated
Do a fresh install from the github install and see that the plugin is there.
The Quicklinks was not in its own group but more hidden.
There was a red image which has been replaced with the icon for information/warning
No warning across the top
The warning message across the top has been restored as in Joomla 2.5 EOS plugin to make it more obvious and allow a place to show different messages over time if needed
The Quicklinks is in its own group and has the icon added and the warning class added.
If the date has passed and the version is 3.10 and its the administrator control panel you should see this
You can simulate this by editing plugins->quicklink->eosnotify->eosnotify.php
line 52 in your instillation
if (!$this->app->isClient('administrator') || version_compare(JVERSION, '3.10', '>=') || Factory::getDate()->toSql() <= '2021-04-15')
and change the date 2021-04-15 to 2021-01-01
to make the plugin think we have passed the nominated date (my birthday)
@zero-24
Please take a look at this draft which I can then add to depending on the outcome of our meeting.
@bayareajenn
Was this more what you had in mind and do we know where the documentation will be held that it needs to link to?
Status | New | ⇒ | Pending |
Category | ⇒ | SQL Administration com_admin Postgresql MS SQL Language & Strings Installation Front End Plugins |
Thanks. Its a draft as we have a meeting in the week to decide on many aspects of the plugin including whether we have dates and whether we want all the message areas. I have updated many aspects of what was there but left the url and text as is. So please dont go too far as this will all change after the meeting in the week
Title |
|
Labels |
Added:
?
?
|
The Warning is not language-aware at all. A sprintf should be used.
A unique string is necessary to cope with that:
PLG_QUICKICON_EOSNOTIFY_CLICK_FOR_INFORMATION_MESSAGE="Joomla 3.10 Support Has Ended. Click <a %s>Here</a> For More Information."
and therefore delete the 3 strings:
PLG_QUICKICON_EOSNOTIFY_CLICK_FOR_INFORMATION_MESSAGE_START="Joomla 3.10 Support Has Ended Click"
PLG_QUICKICON_EOSNOTIFY_CLICK_FOR_INFORMATION_CLICK_WORD="Here"
PLG_QUICKICON_EOSNOTIFY_CLICK_FOR_INFORMATION_MESSAGE_END="for More Information."
Code:
$this->app->enqueueMessage(Text::sprintf('PLG_QUICKICON_EOSNOTIFY_CLICK_FOR_INFORMATION_MESSAGE', 'href="https://www.joomla.org/" target="_blank"'), 'warning');
which gives correctly:
Also It looks like this new string is not used.
PLG_QUICKICON_EOSNOTIFY_MESSAGE="Joomla 3.10 Support Has Ended!!!<br />Click Here for More Information."
@softforge Congratulations on your first PR! Thanks for your service!
The link will go to the same place as the last EOS plugin... docs.joomla.org/Why_Migrate.
As for what I had in mind, I'm going to hold off on answering that part until after the CMS Release Team meets about it this week.
@bayareajenn This is true. I guess I was trying to avoid them having to click on the 3.x to 4.x funnel because in this case we should know exactly what version they are going between. I'm not sure. If you think it's better to send them to the main funnel page I'm fine with that too
@bayareajenn This is true. I guess I was trying to avoid them having to click on the 3.x to 4.x funnel because in this case we should know exactly what version they are going between. I'm not sure. If you think it's better to send them to the main funnel page I'm fine with that too
Leaving it to link to the main funnel just removes some risk that they miss some info in those initial pages.
@softforge You have marked my 2nd review about the code style fix as resolved, but you haven't committed that suggestion or any other fix for the code style error, and so drone is still in failed status.
If you unresolve that conversation you can again use a button to apply the suggested change.
What needs to be done to complete this.
I am assuming that this needs to be merged into 3.10 and that makes it a release blocker for 4.0 at the same time.
As it needed the release dates added to make it work it was finished privately and I gave Tobias the finished package about 3-4 weeks ago. He just needs to review and add the comments the testing team made.
Yes its still on my to do to make a PR.
so can this one be closed?
I dont see why not. Tobias? is there any reason to keep it open? If no then please close
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-06-07 18:10:04 |
Closed_By | ⇒ | zero-24 |
We really should follow best practice and NOT have links that are just called "here".
Can I suggest something like
In addition we are not really providing a good service to our users if we only tell them on the day that support has ended. I would suggest having a progressive series of messages that go from Information to Warning to Error and preferably with an actual date so a user can see something like "will end in 90 days on 1 Nov 2023". I think the code for the php version eol will probably provide you with the suitable code to adopt.