? ? Pending

User tests: Successful: Unsuccessful:

avatar softforge
softforge
2 Jan 2021

Pull Request for Issue # .

Summary of Changes

Updated language files
Updated quicklink plug to add new conditions, add a quicklinks group and restore the warning notice

Testing Instructions

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.

Actual result BEFORE applying this Pull Request

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

Expected result AFTER applying this Pull Request

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)

eos

Documentation Changes Required

@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?

avatar softforge softforge - open - 2 Jan 2021
avatar softforge softforge - change - 2 Jan 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jan 2021
Category SQL Administration com_admin Postgresql MS SQL Language & Strings Installation Front End Plugins
avatar brianteeman
brianteeman - comment - 2 Jan 2021

We really should follow best practice and NOT have links that are just called "here".

Can I suggest something like

PLG_QUICKICON_EOSNOTIFY_CLICK_FOR_INFORMATION_MESSAGE_START=Support for Joomla 3.10 has ended. Please visit
PLG_QUICKICON_EOSNOTIFY_CLICK_FOR_INFORMATION_CLICK_WORD="www.joomla.org"
PLG_QUICKICON_EOSNOTIFY_CLICK_FOR_INFORMATION_MESSAGE_END="for more information."

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.

avatar softforge
softforge - comment - 2 Jan 2021

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

avatar bembelimen bembelimen - change - 3 Jan 2021
Title
3.10/eol plugin
Add the End of Life (EOL) plugin for Joomla! 3.10
avatar bembelimen bembelimen - edited - 3 Jan 2021
avatar softforge softforge - change - 3 Jan 2021
Labels Added: ? ?
avatar infograf768
infograf768 - comment - 4 Jan 2021

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:

Screen Shot 2021-01-04 at 12 40 48

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."

avatar bayareajenn
bayareajenn - comment - 4 Jan 2021

@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.

avatar bayareajenn
bayareajenn - comment - 4 Jan 2021

@wilsonge if we send them directly to the 3.10.x to 4.x planning page, then they won't see the self assessment. Or we can add a link to the self assessment to that page. I'll make a comment on the doc in Drive.

avatar wilsonge
wilsonge - comment - 4 Jan 2021

@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

avatar bayareajenn
bayareajenn - comment - 4 Jan 2021

@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.

avatar richard67
richard67 - comment - 15 Feb 2021

@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.

avatar brianteeman
brianteeman - comment - 7 May 2021

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.

avatar softforge
softforge - comment - 7 May 2021

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.

avatar zero-24
zero-24 - comment - 7 May 2021

Yes its still on my to do to make a PR.

avatar brianteeman
brianteeman - comment - 7 May 2021

so can this one be closed?

avatar softforge
softforge - comment - 8 May 2021

I dont see why not. Tobias? is there any reason to keep it open? If no then please close

avatar zero-24
zero-24 - comment - 7 Jun 2021
avatar zero-24 zero-24 - close - 7 Jun 2021
avatar zero-24 zero-24 - change - 7 Jun 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-06-07 18:10:04
Closed_By zero-24

Add a Comment

Login with GitHub to post a comment