-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[5.2] Add languages API endpoint #42136
Conversation
I have tested this item ✅ successfully on d87a793 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42136. |
I have tested this item ✅ successfully on d87a793 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42136. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42136. |
api/components/com_installer/src/View/Languages/JsonapiView.php
Outdated
Show resolved
Hide resolved
Co-authored-by: Quy <[email protected]>
api/components/com_installer/src/Controller/LanguagesController.php
Outdated
Show resolved
Hide resolved
I have rebased this to 5.1 as it is a new feature. |
Please wait till 5.1 is fully up to date with 5.0. |
this pr is not a feature it'a a bug fix |
For me it looked like a new feature. Next time it would help when you update the pr description with the link. |
I have tested this item ✅ successfully on 7390b4e This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42136. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42136. |
no interest |
Thanks @alikon ! |
Summary of Changes
fix for GET
{{base_path}}/api/index.php/v1/languages
Testing Instructions
GET
{{base_path}}/api/index.php/v1/languages
Actual result BEFORE applying this Pull Request
500 Internal server error
Expected result AFTER applying this Pull Request
200 OK