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.

Sign up for GitHub

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

Fixed put.group_hunt_group method #80

Merged
merged 4 commits into from
Aug 12, 2024
Merged

Fixed put.group_hunt_group method #80

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3873ec9
Fixed put.group_hunt_group method
LivCurtis Jul 26, 2024
4afce27
Updated hunt group methods in put.py
LivCurtis Jul 26, 2024
158072c
Updated put.py
LivCurtis Jul 26, 2024
51031d2
Update put.py
LivCurtis Jul 31, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions odins_spear/methods/put.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -897,10 +897,10 @@ def group_hunt_groups_status(self, hunt_group_user_ids: list, status: bool =True


def group_hunt_group(self, service_provider_id: str, group_id: str, hunt_group_user_id: str, updates: dict):
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change hunt_group_user_id to service_user_id in these args.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done :)

"""Update a Hunt Groups (HG) settings.
"""Update a Hunt Group's (HG) settings.

Args:
hunt_group_user_id (str): Service provider ID of where the group that hosts the HG is located.
service_provider_id (str): Service provider ID of where the group that hosts the HG is located.
group_id (str): Group ID of where the HG is located.
service_user_id (str): Target service user ID of the HG.
updates (dict): Updates to be applied to HG.
Expand All @@ -913,7 +913,8 @@ def group_hunt_group(self, service_provider_id: str, group_id: str, hunt_group_u

updates["serviceProviderId"] = service_provider_id
updates["groupId"] = group_id
updates["serviceUserId"] = hunt_group_user_id
updates["serviceUserId"] = hunt_group_user_id
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should match new argument

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done :)

updates["serviceInstanceProfile"] = {}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check if this is in the updates otherwise here you will override it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done :)


return self.requester.put(endpoint, data=updates)

Expand Down