Skip to content

[4.2] Fixing labels for Smart Search menu item#38228

Merged
roland-d merged 5 commits intojoomla:4.2-devfrom
Hackwar:4.2-finder-language2
Jul 11, 2022
Merged

[4.2] Fixing labels for Smart Search menu item#38228
roland-d merged 5 commits intojoomla:4.2-devfrom
Hackwar:4.2-finder-language2

Conversation

@Hackwar
Copy link
Copy Markdown
Member

@Hackwar Hackwar commented Jul 5, 2022

The language strings used for the 2 request parameter options in the search view of com_finder have translation keys which have a wrong english translation and which are used in 2 different circumstances for different stuff. This adds additional translation keys and "right" translations.

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.2-dev labels Jul 5, 2022
@brianteeman
Copy link
Copy Markdown
Contributor

I have tested this item 🔴 unsuccessfully on f28c99e


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

@brianteeman
Copy link
Copy Markdown
Contributor

please alpha sort language strings

and why add a new string with identical text


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

@Hackwar
Copy link
Copy Markdown
Member Author

Hackwar commented Jul 5, 2022

Because they are used in pretty different situations.

Hackwar and others added 2 commits July 9, 2022 13:37
Co-authored-by: Quy <quy@fluxbb.org>
Co-authored-by: Quy <quy@fluxbb.org>
@chmst
Copy link
Copy Markdown
Contributor

chmst commented Jul 10, 2022

I have tested this item ✅ successfully on 7225f8d

Thanks, this label is much better than before.


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

@chmst chmst added the Maintainers Checked Used if the PR is conceptional useful label Jul 10, 2022
@viocassel
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 7225f8d


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

@chmst chmst removed the Maintainers Checked Used if the PR is conceptional useful label Jul 10, 2022
@chmst
Copy link
Copy Markdown
Contributor

chmst commented Jul 10, 2022

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 10, 2022
@roland-d roland-d merged commit 275ddeb into joomla:4.2-dev Jul 11, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 11, 2022
@roland-d
Copy link
Copy Markdown
Contributor

Thanks everybody

@roland-d roland-d added this to the Joomla! 4.2.0 milestone Jul 11, 2022
heelc29 added a commit to heelc29/joomla that referenced this pull request Jul 13, 2022
Kostelano added a commit to JPathRu/localisation that referenced this pull request Jul 19, 2022
joomla/joomla-cms#36749 +
joomla/joomla-cms#38228 +
joomla/joomla-cms#38227 - (только для en-GB, у нас исправлено ранее)
joomla/joomla-cms#38255 +
joomla/joomla-cms#38244 +
joomla/joomla-cms#38271 - (только для en-GB, у нас исправлено ранее)
joomla/joomla-cms#38287 - (только для en-GB, у нас исправлено ранее)
joomla/joomla-cms#38301 - (только для en-GB, у нас исправлено ранее)
@Hackwar Hackwar deleted the 4.2-finder-language2 branch September 7, 2022 07:43
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.

8 participants