RTC Language Change PR-5.2-dev Guided Tours Pending

User tests: Successful: Unsuccessful:

avatar tecpromotion
tecpromotion
13 Sep 2024

Pull Request for PR #43966

Summary of Changes

Add missing file header
Correction of the spelling of the language strings
add "noreferrer" to link

Testing Instructions

It should have no effect on the functionality.

Actual result BEFORE applying this Pull Request

language_string = ""

Expected result AFTER applying this Pull Request

language_string=""

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 tecpromotion tecpromotion - open - 13 Sep 2024
avatar tecpromotion tecpromotion - change - 13 Sep 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Sep 2024
Category Administration Language & Strings
avatar tecpromotion tecpromotion - change - 13 Sep 2024
The description was changed
avatar tecpromotion tecpromotion - edited - 13 Sep 2024
avatar brianteeman
brianteeman - comment - 13 Sep 2024

Nested Subforms - Allows the nesting of Subform inside a Subform Custom Field.
==>
Nested Subforms - Allows the nesting of a Subform inside a Subform Custom Field.
or
Nested Subforms - Allows the nesting of Subforms inside a Subform Custom Field.

I think its the first option assuming you can only nest one subform otherwise its the second

avatar obuisard
obuisard - comment - 13 Sep 2024

Nested Subforms - Allows the nesting of Subform inside a Subform Custom Field. ==> Nested Subforms - Allows the nesting of a Subform inside a Subform Custom Field. or Nested Subforms - Allows the nesting of Subforms inside a Subform Custom Field.

I think its the first option assuming you can only nest one subform otherwise its the second

The PR #43204 allows the selection of subform fields in a subforms. There could be more than one. So that should be plural.

avatar brianteeman
brianteeman - comment - 14 Sep 2024

Thanks for confirming @obuisard I have suggested the correction now

avatar obuisard obuisard - change - 14 Sep 2024
Labels Added: Language Change PR-5.2-dev Guided Tours
avatar tecpromotion
tecpromotion - comment - 17 Sep 2024

RTC

avatar richard67
richard67 - comment - 17 Sep 2024

RTC

@tecpromotion ??? It doesn't have 2 successful human tests, or am I missing something? Approvals on GitHub just mean the PR is ok by review, but they are not equal to a test of the functionality by a human being.

avatar richard67
richard67 - comment - 17 Sep 2024

Well I just see a review should be sufficient here. But still it needs "testers".

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

I have tested this item ✅ successfully on 4095578

With a good diff tool I was able to compare the details of the changes in the language string, which the diff on GitHub doesn't show.


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

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

I have tested this item ✅ successfully on 4095578


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

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/44075.

avatar Hackwar Hackwar - close - 18 Sep 2024
avatar Hackwar Hackwar - merge - 18 Sep 2024
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:06:39
Closed_By Hackwar
Labels Added: RTC
avatar Hackwar
Hackwar - comment - 18 Sep 2024

Thanks for your contribution @obuisard and @brianteeman

Add a Comment

Login with GitHub to post a comment