-
Notifications
You must be signed in to change notification settings - Fork 558
Fixing various export issues #6019
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
Conversation
Somehow this PR also contains a lot of changes of other PRs, even though those are already merged and available in the Dev branch. |
@ykuijs What you can do is revert the branch to it's original commit you pushed and then do the merge again (or rebase it). You'll most likely need to force push the branch or do the changes again. |
@FabienTschanz, I did a rebase with upstream/Dev to include all changes into my branch. So the original commit no longer is the commit I did before.....since it is now rebased on the most recent commit from Dev. Can you review this PR? Most of the changes are yours anyways, so you already know what those changes are and can skip over them 😉 |
Looks good from what I can tell. I don't know about your changes, but I guess they are ok. |
Add -All to Get-MgServicePrincipalAppRoleAssignedTo - fixes microsoft#5824
Release 1.25.521.1
…cies2 Updated Dependencies
…cyTag---Fix-for-AgeLimitForRetention Update CHANGELOG.md
Release 1.25.528.1
Update Graph.PermissionList.txt
Pull Request (PR) description
This PR fixes various export issues
This Pull Request (PR) fixes the following issues
Task list
Entry should say what was changed and how that affects users (if applicable), and
reference the issue being resolved (if applicable).