Conversation
… translation functions for improved localization and clarity. Updated messages in Get-AbrADInfrastructureService.ps1, Get-AbrADKerberosAudit.ps1, Get-AbrADOU.ps1, Get-AbrADReportBrief.ps1, Get-AbrADSCCM.ps1, Get-AbrADSecurityAssessment.ps1, Get-AbrADSite.ps1, Get-AbrADSiteReplication.ps1, Get-AbrADTrust.ps1, Get-AbrDHCPinAD.ps1, Get-AbrDNSSection.ps1, Get-AbrDomainSection.ps1, and Invoke-DcDiag.ps1.
…d readability and consistency
…zation and documentation clarity
…ram in CHANGELOG and module manifest
…erface fix: Ensure diagram theme generation respects user-selected themes in configuration refactor: Improve multi-language support and documentation clarity for MicrosoftAD.psd1
…ty and user-friendliness Co-authored-by: Copilot <copilot@github.com>
|
PSScriptAnalyzer results: Errors: [0], Warnings: [33], Information: [3]
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR prepares the module for a v1.0.0 public release by introducing a new GUI entry point, tightening localization usage across report/diagram output, and updating module metadata/dependencies to the 1.0.0 release line.
Changes:
- Added a GliderUI-based GUI launcher (
Start-AsBuiltReportMSAD) and exported it via the module manifest/loader. - Refactored many hard-coded user-facing strings to use localized
$reportTranslatekeys (EN/ES) and improved diagram theming behavior. - Bumped module version to 1.0.0 and updated required module versions (AsBuiltReport.Chart / AsBuiltReport.Diagram).
Reviewed changes
Copilot reviewed 53 out of 55 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| copilot-instructions.md | Removes legacy AI instruction document from the repo. |
| README.md | Documents GUI usage and adds a screenshot. |
| CHANGELOG.md | Adds v1.0.0 release notes. |
| AsBuiltReport.Microsoft.AD/AsBuiltReport.Microsoft.AD.psd1 | Bumps module version, updates required module versions, exports Start-AsBuiltReportMSAD. |
| AsBuiltReport.Microsoft.AD/AsBuiltReport.Microsoft.AD.psm1 | Dot-sources GUI scripts and exports GUI function(s). |
| AsBuiltReport.Microsoft.AD/AsBuiltReport.Microsoft.AD.json | Updates default HealthCheck configuration structure. |
| AsBuiltReport.Microsoft.AD/Src/Public/Invoke-AsBuiltReport.Microsoft.AD.ps1 | Version bump + localized console output formatting tweaks. |
| AsBuiltReport.Microsoft.AD/Src/Private/Gui/Start-AsBuiltReportMSAD.ps1 | Adds the GUI implementation for report generation via GliderUI. |
| AsBuiltReport.Microsoft.AD/Src/Private/Report/* | Replaces many hard-coded messages with localized strings; enables DCDiag collection call; minor formatting tweaks. |
| AsBuiltReport.Microsoft.AD/Src/Private/Diagram/* | Diagram theme handling adjustments (edge/subgraph colors, label/signature rendering). |
| AsBuiltReport.Microsoft.AD/Language/en-US/MicrosoftAD.psd1 | Adds/updates localization keys used by the refactors. |
| $List.Add($reportTranslate.GetAbrADSite.DescBP) | ||
| } | ||
| if ($OutObj | Where-Object { $_.$($reportTranslate.GetAbrADSite.Sites) -eq $reportTranslate.GetAbrADSite.NoSiteAssigned }) { | ||
| if ($OutObj | Where-Object { $_.$($reportTranslate.GetAbrADSite.Sites) -eq $reportTranslate.GetAbrADSite.NoSiteAssigned }){ |
Comment on lines
+38
to
+41
| if (-not (Get-Module -ListAvailable -Name GliderUI)) { | ||
| Write-Host 'GliderUI not found — installing from PSGallery…' -ForegroundColor Cyan | ||
| Install-PSResource -Name GliderUI -Version $requiredGliderUIVersion -Scope CurrentUser -TrustRepository | ||
| } |
|
|
||
| **Beta** versions of the GUI may contain bugs and issues. If you encounter any problems while using the GUI, please report them on the project's GitHub Issues page. | ||
|
|
||
|  |
| ### Added | ||
|
|
||
| - Add support for GliderUI graphical interface to generate reports via an intuitive user interface instead of command-line execution | ||
| - Add Windows 10* to the unsupported operating systems list in the Health Check section to reflect the end of support for Windows 10 and encourage users to upgrade to supported operating systems for better security and performance |
Comment on lines
+28
to
29
| Write-PScriboMessage -Message "$($reportTranslate.GetAbrADDCDiag.ErrorInvokeDcDiag -f $DomainController) $($_.Exception.Message)" | ||
| return |
|
|
||
| $TableParams = @{ | ||
| Name = "$($reportTranslate.GetAbrADSite.ConnectionObjects) - $($ForestInfo)" | ||
| Name = "$($reportTranslate.GetAbrADSite.ConnectionObjects)- $($ForestInfo)" |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
PSScriptAnalyzer results: Errors: [0], Warnings: [33], Information: [3]
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[1.0.0] - 2026-05-02
Added
🔃 Changed
This includes improved grammar, punctuation, and readability across various best practice descriptions related to Active Directory configurations
1.0.01.0.70.3.1🐛 Fixed