-
Notifications
You must be signed in to change notification settings - Fork 1
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
New powershell connector (PSv2) #14
Conversation
@rhouthuijzen the code appears to be well-structured. However, since we are focusing on the PS v2 template, please ensure that the code aligns with the template standards. |
This comment was marked as resolved.
This comment was marked as resolved.
Please make sure insite can be set and unset at resp enable & disable The following code should be added
|
This comment was marked as resolved.
This comment was marked as resolved.
Before we add this. We should also make the $previousAccount list dynamic. This needs to be tested first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on what I've seen in the changed/added code, everything is OK. So ready to merge.
tested at custommer environment
Readme is based on the new template
The connector is not based on the new template