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

Rewrite for Powershell and pwsh Core support #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

2mOlaf
Copy link

@2mOlaf 2mOlaf commented Nov 2, 2024

Logic added to control more failure scenarios

  • Checks for admin privileges to install modules
  • Checks for existing Azure contexts before creating new ones
  • If no password is returned, we warn the user that they may not have permissions
  • Changes commands based on Windows Powershell or Powershell Core (pwsh)
  • Created TenantId parameter to pass if user has more than one tenant to which they have access (can create errors without specific selection)
    Old Zero Networks Enterprise App ID changed - added a search to find this instead

Logic added to control more failure scenarios
Remove help text offering SubscriptionName as a variable which was removed prior to commit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant