Skip to content

[4.0] Admin Module Help strings#30458

Merged
wilsonge merged 2 commits intojoomla:4.0-devfrom
brianteeman:testing
Sep 21, 2020
Merged

[4.0] Admin Module Help strings#30458
wilsonge merged 2 commits intojoomla:4.0-devfrom
brianteeman:testing

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

This PR corrects one constant which had been duplicated and provides values for the others.

PR for #30457

This PR corrects one constant which had been duplicated and provides values for the others.

PR for joomla#30457
@ceford
Copy link
Copy Markdown
Contributor

ceford commented Aug 25, 2020

I have tested this item ✅ successfully on 76bf14e

I should have made it clearer for other testers: go to Administrator Modules and set the -Select Position- filter to status. Then select each module in turn and click the edit screen Help button.


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

@ceford
Copy link
Copy Markdown
Contributor

ceford commented Aug 25, 2020

I forgot to check the Login Support module. After applying the patch the Help screen has this in the title bar: https://help.joomla.org/proxy?keyref=Help40:JHELP_EXTENSIONS_MODULE_MANAGER_LOGIN_SUPPORT&lang=en which means the string value is not set: Extensions_Module_Manager_Login_Support - I thought that was being included in this PR.


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

@brianteeman
Copy link
Copy Markdown
Contributor Author

My fault - I made a typo - fixing it now

@ceford
Copy link
Copy Markdown
Contributor

ceford commented Aug 26, 2020

I see the string is fixed. The Help file I created has the wrong name - I need to get a translator admin to do that as I can't once once a file has been marked for translation.


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

@ghost
Copy link
Copy Markdown

ghost commented Sep 7, 2020

I have tested this item ✅ successfully on 40109db


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

1 similar comment
@ChristineWk
Copy link
Copy Markdown

I have tested this item ✅ successfully on 40109db


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

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Sep 21, 2020

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 21, 2020
@wilsonge wilsonge merged commit 8f483ce into joomla:4.0-dev Sep 21, 2020
@wilsonge
Copy link
Copy Markdown
Contributor

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 21, 2020
@wilsonge wilsonge added this to the Joomla 4.0 milestone Sep 21, 2020
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
@brianteeman brianteeman deleted the testing branch November 28, 2020 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants