RTC Language Change NPM Resource Changed PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar obuisard
obuisard
23 Aug 2024

Summary of Changes

This PR adds a tour that highlights new features introduced in Joomla 5.2.

Testing Instructions

Download the update package, not the full install as the tour is only available on updates!

It is possible to find the tour in the list of tours at the top of the Joomla dashboard and run it manually.
Make sure you see images when running the tour (on the first step only) and when editing the tour (System -> Manage -> Guided tours).

Actual result BEFORE applying this Pull Request

No tour highlighting new features for Joomla 5.2.

Expected result AFTER applying this Pull Request

What’s New in Joomla 5.2! tour is available.

image

image

image

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar obuisard obuisard - open - 23 Aug 2024
avatar obuisard obuisard - change - 23 Aug 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Aug 2024
Category SQL Administration com_admin Postgresql Language & Strings Repository NPM Change Front End Plugins
avatar obuisard obuisard - change - 23 Aug 2024
The description was changed
avatar obuisard obuisard - edited - 23 Aug 2024
avatar obuisard obuisard - change - 23 Aug 2024
Labels Added: Language Change NPM Resource Changed PR-5.2-dev
avatar obuisard obuisard - change - 23 Aug 2024
The description was changed
avatar obuisard obuisard - edited - 23 Aug 2024
avatar obuisard obuisard - change - 23 Aug 2024
Title
[5.2] [Guided tours] Add a What's New tour when updating Joomla
[5.2] [Guided tours] Add a What's New tour when updating to Joomla 5.2
avatar obuisard obuisard - edited - 23 Aug 2024
avatar obuisard obuisard - change - 23 Aug 2024
The description was changed
avatar obuisard obuisard - edited - 23 Aug 2024
avatar brianteeman
brianteeman - comment - 23 Aug 2024

Using images in this way means that they are not translatable. Is that a good idea?

avatar brianteeman
brianteeman - comment - 23 Aug 2024

Is the intention for this to replace Post-Installation messages?

avatar Kostelano
Kostelano - comment - 23 Aug 2024

Regarding "untranslatable" pictures - good point. I would prefer that all possible pictures for translation were like that.

Something like the title enhancedtourheader_en-GB.png. If the system does not find a picture with the title enhancedtourheader_YOUR_LANGUAGE_CODE.png, the standard en-GB is loaded.

avatar brianteeman
brianteeman - comment - 23 Aug 2024

Neither the sql for tours or steps adds a value for ordering.

Why does this matter? The module displays tours ordered by ordering. So the whats new is always at the top of the list where it probably should be at the bottom next to the welcome to joomla tour so that it doesnt always display in the dropdown

image

avatar brianteeman
brianteeman - comment - 23 Aug 2024

Regarding "untranslatable" pictures - good point. I would prefer that all possible pictures for translation were like that.

Something like the title enhancedtourheader_en-GB.png. If the system does not find a picture with the title enhancedtourheader_YOUR_LANGUAGE_CODE.png, the standard en-GB is loaded.

not that simple as we dont have a method for you to add an image file into the language pack

avatar obuisard
obuisard - comment - 24 Aug 2024

Is the intention for this to replace Post-Installation messages?

The idea is for the What's new tours to highlight new features of a release (major or minor) on an update, of interest to the user, which is not quite the same as the post-installation messages, which targets admin, pointing that there are changes in htaccess... At least, that is how I see it.

avatar obuisard
obuisard - comment - 24 Aug 2024

Using images in this way means that they are not translatable. Is that a good idea?

No, but we could not find anything to hint us on how to do this (other than how the documentation screens are handled).

avatar obuisard
obuisard - comment - 24 Aug 2024

Neither the sql for tours or steps adds a value for ordering.

Why does this matter? The module displays tours ordered by ordering. So the whats new is always at the top of the list where it probably should be at the bottom next to the welcome to joomla tour so that it doesnt always display in the dropdown

All tours and steps get an ordering value of 0 on installation. Then values change once the user starts ordering tours.
Do you mean we should check all ordering values of the tables, get the highest number, then use that number + 1 in our SQL scripts?

avatar obuisard
obuisard - comment - 24 Aug 2024

Regarding "untranslatable" pictures - good point. I would prefer that all possible pictures for translation were like that.
Something like the title enhancedtourheader_en-GB.png. If the system does not find a picture with the title enhancedtourheader_YOUR_LANGUAGE_CODE.png, the standard en-GB is loaded.

not that simple as we dont have a method for you to add an image file into the language pack

I think adding the language suffix is a good idea, but then we need to add the logic into this PR to find images according to languages. Images ultimately should still remain in the /media folder

avatar brianteeman
brianteeman - comment - 24 Aug 2024

All tours and steps get an ordering value of 0 on installation. Then values change once the user starts ordering tours.

No that is not correct. You can see that the ordering is set on installation here

INSERT INTO `#__guidedtours` (`id`, `title`, `uid`, `description`, `ordering`, `extensions`, `url`, `created`, `created_by`, `modified`, `modified_by`, `checked_out_time`, `checked_out`, `published`, `language`, `access`, `autostart`) VALUES
(1, 'COM_GUIDEDTOURS_TOUR_GUIDEDTOURS_TITLE', 'joomla-guidedtours', 'COM_GUIDEDTOURS_TOUR_GUIDEDTOURS_DESCRIPTION', 1, '["com_guidedtours"]', 'administrator/index.php?option=com_guidedtours&view=tours', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(2, 'COM_GUIDEDTOURS_TOUR_GUIDEDTOURSTEPS_TITLE', 'joomla-guidedtoursteps', 'COM_GUIDEDTOURS_TOUR_GUIDEDTOURSTEPS_DESCRIPTION', 2, '["com_guidedtours"]', 'administrator/index.php?option=com_guidedtours&view=tours', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(3, 'COM_GUIDEDTOURS_TOUR_ARTICLES_TITLE', 'joomla-articles', 'COM_GUIDEDTOURS_TOUR_ARTICLES_DESCRIPTION', 3, '["com_content","com_categories"]', 'administrator/index.php?option=com_content&view=articles', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(4, 'COM_GUIDEDTOURS_TOUR_CATEGORIES_TITLE', 'joomla-categories', 'COM_GUIDEDTOURS_TOUR_CATEGORIES_DESCRIPTION', 4, '["com_content","com_categories"]', 'administrator/index.php?option=com_categories&view=categories&extension=com_content', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(5, 'COM_GUIDEDTOURS_TOUR_MENUS_TITLE', 'joomla-menus', 'COM_GUIDEDTOURS_TOUR_MENUS_DESCRIPTION', 5, '["com_menus"]', 'administrator/index.php?option=com_menus&view=menus', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(6, 'COM_GUIDEDTOURS_TOUR_TAGS_TITLE', 'joomla-tags', 'COM_GUIDEDTOURS_TOUR_TAGS_DESCRIPTION', 6, '["com_tags"]', 'administrator/index.php?option=com_tags&view=tags', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(7, 'COM_GUIDEDTOURS_TOUR_BANNERS_TITLE', 'joomla-banners', 'COM_GUIDEDTOURS_TOUR_BANNERS_DESCRIPTION', 7, '["com_banners"]', 'administrator/index.php?option=com_banners&view=banners', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(8, 'COM_GUIDEDTOURS_TOUR_CONTACTS_TITLE', 'joomla-contacts', 'COM_GUIDEDTOURS_TOUR_CONTACTS_DESCRIPTION', 8, '["com_contact"]', 'administrator/index.php?option=com_contact&view=contacts', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(9, 'COM_GUIDEDTOURS_TOUR_NEWSFEEDS_TITLE', 'joomla-newsfeeds', 'COM_GUIDEDTOURS_TOUR_NEWSFEEDS_DESCRIPTION', 9, '["com_newsfeeds"]', 'administrator/index.php?option=com_newsfeeds&view=newsfeeds', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(10, 'COM_GUIDEDTOURS_TOUR_SMARTSEARCH_TITLE', 'joomla-smartsearch', 'COM_GUIDEDTOURS_TOUR_SMARTSEARCH_DESCRIPTION', 10, '["com_finder"]', 'administrator/index.php?option=com_finder&view=filters', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(11, 'COM_GUIDEDTOURS_TOUR_USERS_TITLE', 'joomla-users', 'COM_GUIDEDTOURS_TOUR_USERS_DESCRIPTION', 11, '["com_users"]', 'administrator/index.php?option=com_users&view=users', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 0),
(12, 'COM_GUIDEDTOURS_TOUR_WELCOMETOJOOMLA_TITLE', 'joomla-welcome', 'COM_GUIDEDTOURS_TOUR_WELCOMETOJOOMLA_DESCRIPTION', 12, '["com_cpanel"]', 'administrator/index.php', CURRENT_TIMESTAMP(), 42, CURRENT_TIMESTAMP(), 42, NULL, NULL, 1, '*', 1, 1);

avatar brianteeman
brianteeman - comment - 24 Aug 2024

Using images in this way means that they are not translatable. Is that a good idea?

No, but we could not find anything to hint us on how to do this (other than how the documentation screens are handled).

I dont have any suggestion to make this work but having english screenshots in an arabic installation is not useful

avatar obuisard
obuisard - comment - 24 Aug 2024

All tours and steps get an ordering value of 0 on installation. Then values change once the user starts ordering tours.

No that is not correct. You can see that the ordering is set on installation here

OMG I should have checked, I forgot about it... I should know, I fixed those SQL lines back in Joomla 4.3 :-)
Anyway, this tour is not installed on a new installation, which means, all ordering will/may be different once we add this tour (and many more can have been added in between). So I don't see what ordering should be set here.

avatar obuisard
obuisard - comment - 24 Aug 2024

Using images in this way means that they are not translatable. Is that a good idea?

No, but we could not find anything to hint us on how to do this (other than how the documentation screens are handled).

I dont have any suggestion to make this work but having english screenshots in an arabic installation is not useful

I agree completely.
I put this PR in draft so we can stir some ideas about that problematic...

avatar brianteeman
brianteeman - comment - 24 Aug 2024

It could be argued that the two screenshots do not add anything to the guided tour and can simply be removed.

avatar obuisard obuisard - change - 24 Aug 2024
The description was changed
avatar obuisard obuisard - edited - 24 Aug 2024
avatar obuisard obuisard - change - 30 Aug 2024
The description was changed
avatar obuisard obuisard - edited - 30 Aug 2024
avatar obuisard obuisard - change - 30 Aug 2024
The description was changed
avatar obuisard obuisard - edited - 30 Aug 2024
avatar exlemor exlemor - test_item - 3 Sep 2024 - Tested successfully
avatar exlemor
exlemor - comment - 3 Sep 2024

I have tested this item ✅ successfully on 97b4ef3

I have tested this successfully.. everything worked great.

NOTE: on the 2nd image, the envelope icon is hard to see for someone that sees perfectly well so definitely not good enough for those with visual impairments.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43966.

avatar obuisard
obuisard - comment - 4 Sep 2024

NOTE: on the 2nd image, the envelope icon is hard to see for someone that sees perfectly well so definitely not good enough for those with visual impairments.

Thank you for testing Emmanuel @exlemor! I have updated the image in question.

avatar obuisard obuisard - change - 4 Sep 2024
The description was changed
avatar obuisard obuisard - edited - 4 Sep 2024
avatar obuisard obuisard - change - 4 Sep 2024
The description was changed
avatar obuisard obuisard - edited - 4 Sep 2024
avatar fgsw
fgsw - comment - 4 Sep 2024

Using prebuilt package no "Whats new in Joomla 5.2" available:

Untitled1

"Show all tours":

Untitled2
index.php?option=com_guidedtours&view=tours shows 12 published and no unpublished tours.

avatar obuisard obuisard - change - 4 Sep 2024
The description was changed
avatar obuisard obuisard - edited - 4 Sep 2024
avatar obuisard
obuisard - comment - 4 Sep 2024

Using prebuilt package no "Whats new in Joomla 5.2" available:

Hello @fgsw, you have to install the update package, you cannot use the full install package.
When on new installs, you get the Welcome tour.
On updated site, you get the What's new tour.

I have updated the test section to add that information (sorry, it was missing).

Thanks for trying the PR out!

avatar fgsw fgsw - test_item - 6 Sep 2024 - Tested successfully
avatar fgsw
fgsw - comment - 6 Sep 2024

I have tested this item ✅ successfully on 97b4ef3


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43966.

avatar obuisard obuisard - change - 6 Sep 2024
The description was changed
avatar obuisard obuisard - edited - 6 Sep 2024
avatar exlemor exlemor - test_item - 6 Sep 2024 - Tested successfully
avatar exlemor
exlemor - comment - 6 Sep 2024

I have tested this item ✅ successfully on 97b4ef3

I have re-tested this successfully with the latest PR Update.zip from ~17:26 UTC of today.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43966.
avatar Hackwar Hackwar - change - 11 Sep 2024
Status Pending Ready to Commit
avatar Hackwar
Hackwar - comment - 11 Sep 2024

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43966.

avatar pe7er pe7er - change - 11 Sep 2024
Labels Added: RTC
avatar pe7er pe7er - change - 12 Sep 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-09-12 03:43:49
Closed_By pe7er
avatar pe7er pe7er - close - 12 Sep 2024
avatar pe7er pe7er - merge - 12 Sep 2024
avatar pe7er
pe7er - comment - 12 Sep 2024

Thanks @obuisard !

Add a Comment

Login with GitHub to post a comment