Skip to content

Add new module - Invoke-AzureADUsersAttributes#49

Closed
godylockz wants to merge 3 commits into
dafthack:mainfrom
godylockz:main
Closed

Add new module - Invoke-AzureADUsersAttributes#49
godylockz wants to merge 3 commits into
dafthack:mainfrom
godylockz:main

Conversation

@godylockz
Copy link
Copy Markdown

No description provided.

@dafthack
Copy link
Copy Markdown
Owner

dafthack commented Apr 7, 2026

Thanks for the contribution. I reviewed this PR and I’m going to close it rather than merge it as-is.

The main issue is that the actual changes don’t line up cleanly with the PR title. I wasn’t able to clearly identify a self-contained Invoke-AzureADUsersAttributes addition, while the diff also includes a large amount of churn in GraphRunner.ps1 plus a separate GUI behavior change in GraphRunnerGUI.html that changes email recipient handling from toRecipients to bccRecipients.

Because this repo is used heavily for labs and compatibility matters a lot, I try to avoid merging large PRs that mix unrelated behavior changes with broad file churn. If you want to resubmit this, I’d be happy to review a much smaller PR that contains only the new PowerShell module, minimizes unrelated formatting/whitespace changes, and keeps GUI changes separate.

@dafthack dafthack closed this Apr 7, 2026
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.

2 participants