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.

Sign up for GitHub

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

Jump to bottom

Fix a few issues with AD cmdlets #6527

Merged
merged 3 commits into from
Jun 25, 2018
Merged

Fix a few issues with AD cmdlets #6527

merged 3 commits into from
Jun 25, 2018

Conversation

cormacpayne
Copy link
Member

@cormacpayne cormacpayne commented Jun 25, 2018 •
edited
Loading

Description

Checklist

@cormacpayne
Fix the following issues in AD cmdlets:
1e87ce4 
- Add-AzureRmADGroupMember never worked due to incorrect URL
- New-AzureRmADAppCredential ignore -EndDate parameter if provided
- Remove-AzureRmADGroupMember cmdlet no longer has mandatory -PassThru parameter
@cormacpayne
Update change log for Resources
da93ae4
@cormacpayne
Update help files in Resources
6752dd7
@maddieclayton maddieclayton self-assigned this Jun 25, 2018
@maddieclayton
Copy link
Contributor

on demand here: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/powershell-demand/958/

@cormacpayne cormacpayne merged commit 11e984c into Azure:preview Jun 25, 2018
@AndyHerb AndyHerb mentioned this pull request Jun 29, 2018
Closed
@cormacpayne cormacpayne deleted the fix-add-member branch September 12, 2018 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maddieclayton maddieclayton maddieclayton approved these changes

Assignees

@maddieclayton maddieclayton

Labels
Waiting for On Demand
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cormacpayne @maddieclayton @azuresdkci