-
-
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
127 auto attendant functionality #128
Conversation
Added delete.auto_attendant method
Added auto_attendant_submenu method into delete.py
Added auto_attendant_user to get.py
Added auto_attendant_submenus method to get.py
Added auto_attendant_submenu_usage method to get.py
Added auto_attendant method to post.py
Added auto_attendant_remove_user and auto_attendant_submenu methods into post.py
Corrected typo
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.
Some changes regarding the payload Liv in the post method.
""" | ||
|
||
endpoint = "/groups/auto-attendants" | ||
|
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.
This needs update the existing payload otherwise your overwriting whatever is passed in the param.
Example:
payload['groupId'] = group_id
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.
Additionally things where you have hard coded like "nameDialingEntries":"LastName + FirstName" we need to check this is empty and not passed in by the user before editing.
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.
I've resolved this and pushed new commits.
Corrected typo in post.auto_attendant_submenu method
Updates post.auto_attendant method to ensure payload is not overwritten when passed as a parameter
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.
All good, nice work Liv!!
Added new methods: