Pull Request: Add Azure AD App Configuration Documentation#23
Pull Request: Add Azure AD App Configuration Documentation#23aollivierre wants to merge 8 commits intoSorenLundt:mainfrom
Conversation
Fix Client ID
I've created a comprehensive `AZURE_APP_CONFIGURATION.md` file that covers: 1. Background and reasons for using a custom app ID 2. Step-by-step instructions for creating and configuring the Azure AD application 3. How to update the scripts with the new app ID 4. Common troubleshooting issues and solutions 5. Best practices for security and maintenance 6. Required roles and permissions 7. Additional resources for further reading The documentation is structured to be both beginner-friendly and detailed enough for experienced administrators. It includes all the issues we encountered and their solutions, as well as additional potential problems users might face.
There was a problem hiding this comment.
@InnovationForge-com
Hi, and sorry for no input on your great work here!
I see that in general this commit has a lot of diff lines due to changing from using " to use ' in most code.
Could you review this only change relevant parts ?
This would make it easier to review the actual code changed and be able to merge into main branch
There was a problem hiding this comment.
Hey,
I changed ' back to " with Claude in VSC. Looks good to me.
I won't change back the improved spacing formatting though. Because of this there are still many changes shown that are not actually code improvements.
There was a problem hiding this comment.
I removed in my fork the .exe and put a .cmd instead.
Copied it from https://github.com/Micke-K/IntuneManagement. So should be working for everyone.
There was a problem hiding this comment.
@InnovationForge-com
Hi, and sorry for no input on your great work here!
I see that in general this commit has a lot of diff lines due to changing from using " to use ' in most code.
Could you review this only change relevant parts ?
This would make it easier to review the actual code changed and be able to merge into main branch
There was a problem hiding this comment.
Hey,
I changed ' back to " with Claude in VSC. Looks good to me.
I won't change back the improved spacing formatting though. Because of this there are still many changes shown that are not actually code improvements.
3 participants
Description
This PR adds detailed documentation for Azure AD application configuration, helping users avoid authentication issues and providing a more secure way to interact with Microsoft Graph API.
Changes Made
Added new file:
AZURE_APP_CONFIGURATION.mdModified
WinGet-WrapperImportFromCSV.ps1Why is this needed?
Recent changes in Microsoft's infrastructure and security policies have made it necessary to provide clear guidance on:
Testing Done
Checklist
Related Issues
Closes #[Issue number] (Reference to the issue about authentication problems)
Screenshots
[You can add screenshots of the Azure portal configuration steps here]