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.

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

[Feature Request]: Add GDAP Roles Viva Goals Administrator and Viva Pulse Administrator #2396

Closed
peter-fisher opened this issue May 2, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request no-priority

Comments

@peter-fisher
Copy link
Contributor

Description of the new feature - must be an in-depth explanation of the feature you want, reasoning why, and the added benefits for MSPs as a whole.

Noticed that Viva Goals Administrator and Viva Pulse Administrator roles were missing from the GDAP options. I just added them to the two JSON files in my repo and it seems to work fine.

Here is the updated JSON file -
GDAPRoles.json

PowerShell commands you would normally use to achieve above request

No response

@peter-fisher peter-fisher added enhancement New feature or request no-priority labels May 2, 2024
@MWGMorningwood
Copy link
Contributor

Why not make a PR to dev?

@peter-fisher
Copy link
Contributor Author

Why not make a PR to dev?

I've never contributed before and the CIPP docs say to create an issue first. I'm also not well-versed in making pull requests to be honest. When I attempted the PR, it looked like it was going to also include a commit for the github workflow yml file and I didn't know how to take that out.

@MWGMorningwood
Copy link
Contributor

Check out https://docs.cipp.app/dev-documentation/cipp-dev-guide/setting-up-for-local-development
.gitignore is how you'd exclude specific files, but making a local dev env solves the problem indirectly.

@peter-fisher
Copy link
Contributor Author

Check out https://docs.cipp.app/dev-documentation/cipp-dev-guide/setting-up-for-local-development .gitignore is how you'd exclude specific files, but making a local dev env solves the problem indirectly.

Thanks. I started that process. For now, it looks like if I just submit the PR from the dev branch instead of main it doesn't include any workflow files so I'll do that now.

@peter-fisher
Copy link
Contributor Author

I would like to work on this please!

Copy link

github-actions bot commented May 3, 2024

Great! I assigned you (@peter-fisher) to the issue. Have fun working on it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request no-priority
Projects
None yet
Development

No branches or pull requests

3 participants