Skip to content

Comments

Create a users group through webservices API - fix for Issue #41113#41141

Merged
obuisard merged 2 commits intojoomla:4.3-devfrom
MacJoom:UsersGroupAPIIssue
Sep 6, 2023
Merged

Create a users group through webservices API - fix for Issue #41113#41141
obuisard merged 2 commits intojoomla:4.3-devfrom
MacJoom:UsersGroupAPIIssue

Conversation

@MacJoom
Copy link
Contributor

@MacJoom MacJoom commented Jul 11, 2023

Pull Request for Issue #41113.

Summary of Changes

delete required="true" from id field

Testing Instructions

See
#41113

do a webservices api call

curl --location '{{base_url}}api/index.php/v1/users/groups' --header 'Content-Type: application/json' --data '{"title": "Group Title", "parent_id": 1}'

Actual result BEFORE applying this Pull Request

Response Code: 400 Bad Request - { "errors": [ { "title": "Field required: id" } ] }

Expected result AFTER applying this Pull Request

Works, created group is returned

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

@alikon
Copy link
Contributor

alikon commented Aug 20, 2023

I have tested this item ✅ successfully on 0a946ad


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

1 similar comment
@Quy
Copy link
Contributor

Quy commented Sep 5, 2023

I have tested this item ✅ successfully on 0a946ad


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

@Quy Quy removed the bug label Sep 5, 2023
@Quy
Copy link
Contributor

Quy commented Sep 5, 2023

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 5, 2023
@Quy Quy added the bug label Sep 5, 2023
@obuisard obuisard added this to the Joomla! 4.3.5 milestone Sep 6, 2023
@obuisard obuisard merged commit f470264 into joomla:4.3-dev Sep 6, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 6, 2023
@obuisard
Copy link
Contributor

obuisard commented Sep 6, 2023

Thanks Martin @MacJoom for the PR :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants