Skip to content
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

allow service principals and nested groups to be returned in membership attributes #1507

Merged
merged 8 commits into from
May 21, 2024

Conversation

kent007
Copy link
Contributor

@kent007 kent007 commented Mar 21, 2024

SUMMARY

Fixes #1506

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

azure_rm_adgroup
azure_rm_adgroup_info

ADDITIONAL INFORMATION

I also fixed a few typos in docs and function names. Verified with some manual testing and a clean pass of ansible sanity tests

@kent007 kent007 changed the title allow service_principals and nested groups to be returned in membership attributes allow service principals and nested groups to be returned in membership attributes Mar 21, 2024
plugins/modules/azure_rm_adgroup_info.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_adgroup_info.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_adgroup.py Outdated Show resolved Hide resolved
@Fred-sun Fred-sun added question Further information is requested medium_priority Medium priority work in In trying to solve, or in working with contributors labels Mar 22, 2024
Co-authored-by: Fred-sun <37327967+Fred-sun@users.noreply.github.com>
@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged and removed work in In trying to solve, or in working with contributors labels Mar 25, 2024
@xuzhang3 xuzhang3 merged commit 8125529 into ansible-collections:dev May 21, 2024
Justwmz pushed a commit to Justwmz/azure that referenced this pull request Nov 4, 2024
…ip attributes (ansible-collections#1507)

* commit WIP group stuff

* Fix: group returns

* remove errant parameter

* clean up docs

* fix doc

* remove underscore

* Apply suggestions from code review

Co-authored-by: Fred-sun <37327967+Fred-sun@users.noreply.github.com>

---------

Co-authored-by: Fred-sun <37327967+Fred-sun@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
medium_priority Medium priority question Further information is requested ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

allow service_principals and groups to appear in group_members output
3 participants