Add functions to disable additional AD accounts #90
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the DisableADAccountsMenu function to target different AD admin accounts. The changes replace the existing account names (Kurtis Marsden, Jamie Beale, Luke Leigh) with new ones (John Smith, Jack Daniels, Geoff Geoffries) while maintaining the same functional structure.
- Updates function names to match new admin accounts
- Changes AD account identities in Disable-ADAccount commands
- Updates menu display text and switch statement mappings
| function Disable-JohnSmith { | ||
| Disable-ADAccount -Identity JohnSmith.admin | ||
| Write-Output "Account JohnSmith.admin has been disabled." |
There was a problem hiding this comment.
The function name 'Disable-JohnSmith' doesn't follow PowerShell naming conventions. Consider using 'Disable-JohnSmithAccount' to be more descriptive and consistent with the action being performed.
| function Disable-JackDaniels { | ||
| Disable-ADAccount -Identity JackDaniels.admin | ||
| Write-Output "Account JackDaniels.admin has been disabled." |
There was a problem hiding this comment.
The function name 'Disable-JackDaniels' doesn't follow PowerShell naming conventions. Consider using 'Disable-JackDanielsAccount' to be more descriptive and consistent with the action being performed.
| function Disable-GeoffGeoffries { | ||
| Disable-ADAccount -Identity GeoffGeoffries.admin | ||
| Write-Output "Account GeoffGeoffries.admin has been disabled." |
There was a problem hiding this comment.
The function name 'Disable-GeoffGeoffries' doesn't follow PowerShell naming conventions. Consider using 'Disable-GeoffGeoffriesAccount' to be more descriptive and consistent with the action being performed.
No description provided.