RTC Language Change Release Blocker PR-5.2-dev Guided Tours Pending

User tests: Successful: Unsuccessful:

avatar obuisard
obuisard
15 Sep 2024

Pull Request for Issue #44080.

Summary of Changes

The Guided Tours component, until now, did not have a dispatcher.
We do not want to have restrictions on the Ajax call when a tour is running so that any user having access to the administrator console can have its tour state recorder.

Testing Instructions

Create a user with Manager access permissions.
Log as that user.
Either the welcome or the what's new tour should start.

Actual result BEFORE applying this Pull Request

When dismissing the tour (in any way), you get the error:
image
The state of the tour is not recorded.

Expected result AFTER applying this Pull Request

No more errors.
On an auto-starting tour, the state of the tour is recorded in #__user_profiles table.
If tours are logged, there should be entries in the action logs.

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 - 15 Sep 2024
avatar obuisard obuisard - change - 15 Sep 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Sep 2024
Category Administration
a71f1d4 15 Sep 2024 avatar obuisard Typo
avatar obuisard obuisard - change - 15 Sep 2024
Labels Added: PR-5.2-dev Guided Tours
avatar obuisard obuisard - change - 15 Sep 2024
The description was changed
avatar obuisard obuisard - edited - 15 Sep 2024
avatar obuisard obuisard - change - 15 Sep 2024
Title
[5.2] [Guided Tours]
[5.2] [Guided Tours] Fix permission errors when recording user tour state
avatar obuisard obuisard - edited - 15 Sep 2024
avatar obuisard obuisard - change - 15 Sep 2024
The description was changed
avatar obuisard obuisard - edited - 15 Sep 2024
avatar obuisard obuisard - change - 15 Sep 2024
The description was changed
avatar obuisard obuisard - edited - 15 Sep 2024
avatar obuisard obuisard - change - 15 Sep 2024
The description was changed
avatar obuisard obuisard - edited - 15 Sep 2024
avatar obuisard obuisard - change - 16 Sep 2024
Labels Added: Release Blocker
avatar joomla-cms-bot joomla-cms-bot - change - 16 Sep 2024
Category Administration Administration Language & Strings
avatar obuisard obuisard - change - 16 Sep 2024
Labels Added: Language Change
avatar brianteeman brianteeman - test_item - 16 Sep 2024 - Tested successfully
avatar brianteeman
brianteeman - comment - 16 Sep 2024

I have tested this item ✅ successfully on 9af535f

Issue replicated and fix confirmed


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

avatar richard67 richard67 - alter_testresult - 16 Sep 2024 - brianteeman: Tested successfully
avatar richard67
richard67 - comment - 16 Sep 2024

I've restored @brianteeman 's previous test result in the issue tracker as the change after that was only a case change in a language string.

avatar tecpromotion tecpromotion - test_item - 17 Sep 2024 - Tested successfully
avatar tecpromotion
tecpromotion - comment - 17 Sep 2024

I have tested this item ✅ successfully on 8b919f5


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

avatar tecpromotion
tecpromotion - comment - 17 Sep 2024

RTC

avatar richard67 richard67 - change - 17 Sep 2024
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 17 Sep 2024

RTC


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

avatar Quy Quy - change - 18 Sep 2024
Labels Added: RTC
avatar Hackwar Hackwar - change - 18 Sep 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-09-18 21:14:19
Closed_By Hackwar
avatar Hackwar Hackwar - close - 18 Sep 2024
avatar Hackwar Hackwar - merge - 18 Sep 2024
avatar Hackwar
Hackwar - comment - 18 Sep 2024

Thank you for your contribution @obuisard!

Add a Comment

Login with GitHub to post a comment