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

feat(APIConnector): update unpairing agent method #30

Merged
merged 5 commits into from
Jul 30, 2024
Merged

feat(APIConnector): update unpairing agent method #30

merged 5 commits into from
Jul 30, 2024

Conversation

cpainchaud
Copy link
Owner

Updated the method for unpairing agents in the APIConnector class. Splitting unpairing requests into batches of 500 and added error handling to mark errors for unknown hrefs. Also updated the objects_workload_unpair_multi method to replace '/workloads/' with '/vens/' in the hrefs and changed the API endpoint to '/vens/unpair'. Added new JSON payload types for the updated unpairing method.

… compliance

Updated code in the workload_import.py file to enhance readability and efficiency. Changes include removal of unnecessary whitespace, improved formatting for better readability. Also, removed unused 'Union' import from the typing module.
Refactored code to improve readability and compliance with PEP 8 standards. This includes removing unnecessary whitespace, improving line alignment, and adding spaces around operators and after commas where needed. Also corrected some type hinting issues. No changes were made to the logic or functionality of the code.
Updated the method for unpairing agents in the APIConnector class. Splitting unpairing requests into batches of 500 and added error handling to mark errors for unknown hrefs. Also updated the objects_workload_unpair_multi method to replace '/workloads/' with '/vens/' in the hrefs and changed the API endpoint to '/vens/unpair'. Added new JSON payload types for the updated unpairing method.
@cpainchaud cpainchaud merged commit 2fa738f into master Jul 30, 2024
3 checks passed
This pull request was closed.
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.

1 participant