Feature RTC PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar rdeutz
rdeutz
11 May 2024

Pull Request for Issue #34609 .

Summary of Changes

Adding an option to the newsflash module to show, hide use global setting for the text at the read more button

Testing Instructions

  1. Read the issue and confirm the issue on Joomla 5.2 with sample data
  2. apply patch
  3. go to the "Latest Posts" module
  4. change the "Read More with Title" options in the module and check if the hide/show have the effect we want
  5. change the "Read More with Title" options in the module to "Use global article settings"
  6. go to content -> options and change under articles the Read More with Title and check if the hide/show have the effect we want

Actual result BEFORE applying this Pull Request

See issue

Expected result AFTER applying this Pull Request

You can control from the newflash module what text is at the read more button in the newsflash module

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 rdeutz rdeutz - open - 11 May 2024
avatar rdeutz rdeutz - change - 11 May 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 May 2024
Category Language & Strings Modules Front End
avatar rdeutz rdeutz - change - 13 May 2024
Labels Added: Language Change PR-5.2-dev
avatar KingLouis1 KingLouis1 - test_item - 15 Jul 2024 - Not tested
avatar KingLouis1
KingLouis1 - comment - 15 Jul 2024

I have not tested this item.

Not able to test, as instruction not clear. Where ist this module?


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

avatar crimle
crimle - comment - 15 Jul 2024

I am bewildered. First you mention the «newsflash module». Later you ask «go to the "Latest Posts" module». I was unable to test thoroughly.


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

avatar Quy
Quy - comment - 15 Jul 2024

Please test with the Articles - Newsflash module.

avatar crimle crimle - test_item - 16 Jul 2024 - Tested successfully
avatar crimle
crimle - comment - 16 Jul 2024

I have tested this item ✅ successfully on 0a2af87

Should be
3. go to the "Articles - Newsflash" module
NOT 3. go to the "Latest Posts" module
then the testing instructions are comprehensible. My tests were successful.


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

avatar viocassel viocassel - test_item - 22 Jul 2024 - Tested successfully
avatar viocassel
viocassel - comment - 22 Jul 2024

I have tested this item ✅ successfully on 0a2af87


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

avatar richard67
richard67 - comment - 3 Aug 2024

@rdeutz Could you check the changes suggested by @Quy ?

avatar pcombet-adosis
pcombet-adosis - comment - 22 Aug 2024

Hello,
tested, it does work ✅
I have tested it in a Multi language environment (FR/ENG)
All clear
Best
Phil

avatar richard67
richard67 - comment - 22 Aug 2024

Hello, tested, it does work ✅ I have tested it in a Multi language environment (FR/ENG) All clear Best Phil

@pcombet-adosis Please go to the issue tracker here https://issues.joomla.org/tracker/joomla-cms/43457 and mark your test result by using the blue "Test this" button at the top left corner, selecting your test result and finally submit, so that your test is properly counted. Just a comment with a green check mark is not enough. Thanks in advance.

avatar pcombet-adosis pcombet-adosis - test_item - 22 Aug 2024 - Tested successfully
avatar pcombet-adosis
pcombet-adosis - comment - 22 Aug 2024

I have tested this item ✅ successfully on 0a2af87

I have tested it in a Multi language environment (FR/ENG)
All clear

Best
Philippe


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

avatar crommie crommie - test_item - 24 Aug 2024 - Tested successfully
avatar crommie
crommie - comment - 24 Aug 2024

I have tested this item ✅ successfully on 0a2af87

Works as described.


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

avatar softforge softforge - test_item - 24 Aug 2024 - Tested successfully
avatar softforge
softforge - comment - 24 Aug 2024

I have tested this item ✅ successfully on 0a2af87

Did what it says on the tin


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

avatar rdeutz rdeutz - change - 24 Aug 2024
Labels Added: Updates Requested
avatar joomla-cms-bot joomla-cms-bot - change - 24 Aug 2024
Category Language & Strings Modules Front End Modules Front End
avatar richard67 richard67 - change - 24 Aug 2024
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 24 Aug 2024

As the last change after the successful human tests did just change the language string to the already existing global string, which is already used elsewhere in the same XML, which I've just checked by review, the previous human tests are still valid => RTC.


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

avatar richard67 richard67 - change - 24 Aug 2024
Labels Added: RTC
Removed: Language Change Updates Requested
avatar richard67 richard67 - alter_testresult - 24 Aug 2024 - crimle: Tested successfully
avatar richard67 richard67 - alter_testresult - 24 Aug 2024 - viocassel: Tested successfully
avatar richard67 richard67 - alter_testresult - 24 Aug 2024 - pcombet-adosis: Tested successfully
avatar richard67 richard67 - alter_testresult - 24 Aug 2024 - crommie: Tested successfully
avatar richard67 richard67 - alter_testresult - 24 Aug 2024 - softforge: Tested successfully
avatar HLeithner
HLeithner - comment - 2 Sep 2024

This pull request has been automatically rebased to 5.3-dev.

avatar HLeithner HLeithner - change - 2 Sep 2024
Title
[5.2]Add option for show/hide title in readmore
[5.3] Add option for show/hide title in readmore
avatar HLeithner HLeithner - edited - 2 Sep 2024
avatar rdeutz rdeutz - change - 10 Oct 2024
Labels Added: Feature PR-5.3-dev
Removed: PR-5.2-dev
avatar drmenzelit drmenzelit - change - 16 Oct 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-10-16 13:41:31
Closed_By drmenzelit
avatar drmenzelit drmenzelit - close - 16 Oct 2024
avatar drmenzelit drmenzelit - merge - 16 Oct 2024

Add a Comment

Login with GitHub to post a comment