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

Updated Reports #86

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Updated Reports #86

wants to merge 1 commit into from

Conversation

byteben
Copy link
Contributor

@byteben byteben commented Feb 25, 2025

Report Script Improvements

Pull Request Type

  • New Script
  • Edit Scripts
  • Repository Improvement

Brief summary of changes

Fixed issue in Get-IntuneReport.ps1. If the folder doesn't exist already we create it now. We also check if the PowerShell module is already installed instead of trying to import it each time the script is ran.
Updated pricing in Get-PMPCFoundApps.ps1 and added the ability to ignore prefixes in app names using the array variable [string[]]$IgnorePrefixes = @(),

Checklist

  • Did you Clean your script - No environment details, comments are PG-13
  • Did you test your script
  • If required, did you create, and include a Read Me as outlined in Community Scripts Read Me

Notes for PMPC Team

Fixed issue in Get-IntuneReport.ps1. If the folder doesn't exists already we create it.
Updated pricing in Get-PMPCFoundApps.ps1 and added the ability to ignore prefixes in app names
@byteben byteben requested a review from a team as a code owner February 25, 2025 10:34
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