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

Fix up missing common AD parameters on some cmdlets #61

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

jborean93
Copy link
Collaborator

SUMMARY

Two cmdlets were missing the @adParams splat which contained the domain_server and domain_username/password values. This PR adds that splatted variable to those calls.

Fixes: #54

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

microsoft.ad._ADObject

@softwarefactory-project-zuul
Copy link

@softwarefactory-project-zuul
Copy link

@jborean93 jborean93 merged commit a89e2b0 into ansible-collections:main Aug 7, 2023
19 checks passed
@jborean93 jborean93 deleted the move-adparams branch August 7, 2023 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing @adParams for Rename-ADObject and Move-ADObject
1 participant