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

Update Add-TeamViewerManager.ps1 #13

Merged
merged 7 commits into from
Apr 25, 2023
Merged

Conversation

OtterKring
Copy link
Contributor

included adding UserGroups as manager.

included adding UserGroups as manager
@danieljoos
Copy link
Contributor

danieljoos commented Oct 7, 2022

💙

Good to see that the Powershell module gets someone to work on 😄.

Would be great if you could also update the docs:

Import-Module .\TeamViewerPS
Update-MarkdownHelp .\docs\commands

Revert all changes made by that command except those to Add-TeamViewerManager docs. Then document the new CLI attributes.

@OtterKring
Copy link
Contributor Author

In der Not frißt der Teufel Fliegen. ;-P

Ok, updated.

@danieljoos
Copy link
Contributor

lgtm (but I can't merge anylonger). @koutsop

@koutsop
Copy link
Contributor

koutsop commented Oct 14, 2022

Hi @danieljoos,

Unfortunately, I don't have write access, thus I cannot approve and run the workflow. @ChristianJ-TV do you may have write access?

@MarcelRaad
Copy link
Member

MarcelRaad commented Oct 14, 2022

@koutsop The @teamviewer/tensor team has write access now :-)

@OtterKring
Copy link
Contributor Author

I added the Remove-TeamViewerUser function. Function, Test and Help .md are based on Remove-TeamViewerGroup.

@danieljoos
Copy link
Contributor

To me the Remove-TeamViewerUser should go into its own PR.

@ChristianJ-TV ChristianJ-TV merged commit 4d41a1f into teamviewer:main Apr 25, 2023
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.

5 participants