Skip to content

Conversation

@codevfahad
Copy link
Contributor

@codevfahad codevfahad commented Oct 31, 2025

Added a ‘Permissions’ section with minimal delegated and application permissions
for the m365 entra groupsettingtemplate get command.

Closes #6938

…ommand

Added a ‘Permissions’ section with minimal delegated and application permissions 
for the m365 entra groupsettingtemplate get command.
@codevfahad
Copy link
Contributor Author

Hi @Jwaegebaert and team 👋

As part of Hacktoberfest 2025, I’ve added the Permissions section for the m365 entra groupsettingtemplate get command.

The section includes minimal delegated and application permissions, following the documentation standard used across other commands.

I noticed the issue is open and unassigned, so I went ahead and created this PR to help wrap it up before the Hacktoberfest deadline.

Please review and let me know if any adjustments are needed. 😊

@MartinM85 MartinM85 added the docs label Oct 31, 2025
@MartinM85
Copy link
Contributor

Hi @codevfahad, thank you for the contribution 🚀.

Please check the documentation how the permissions should be formatted.

Could you please add the permissions section also for the entra groupsettingtemplate list command as mentioned in #6938?

@codevfahad
Copy link
Contributor Author

Thanks @MartinM85 — I’ve updated the permissions formatting to use the table layout as per the docs, and added the section to the groupsettingtemplate list command as well. Please review and let me know if any further updates are needed. 😊

@MartinM85 MartinM85 self-assigned this Oct 31, 2025
Copy link
Contributor

@MartinM85 MartinM85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice contribution @codevfahad 🚀. I have a couple of small comments, nothing major.

@MartinM85 MartinM85 marked this pull request as draft October 31, 2025 15:17
@MartinM85 MartinM85 added the hacktoberfest-accepted Accept for hacktoberfest, will merge later label Oct 31, 2025
@codevfahad
Copy link
Contributor Author

Thanks @MartinM85 ! I’ve updated both files to use the least privileged permission GroupSettings.Read.All and improved the table formatting as suggested. 😊

@MartinM85 MartinM85 marked this pull request as ready for review November 1, 2025 08:32
Copy link
Contributor

@MartinM85 MartinM85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. I will merge it later.

@MartinM85
Copy link
Contributor

Merged manually.

@MartinM85 MartinM85 closed this Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs hacktoberfest-accepted Accept for hacktoberfest, will merge later pr-merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Minimal permissions for 'entra groupsettingtemplate' commands

2 participants