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

Breaking changes #154

Merged
merged 2 commits into from
Oct 5, 2023
Merged

Breaking changes #154

merged 2 commits into from
Oct 5, 2023

Conversation

rwiker
Copy link
Contributor

@rwiker rwiker commented Oct 4, 2023

Breaking changes:

  • SumoClient .get() and .delete() no longer decode the response;
    instead, the response object is returned.
  • Instead of keyword parameters, SumoClient .get() takes the parameter
    params, which is a dict.
  • The parameter params is also added to a few other methods. In
    particular, it is now possible to pass a id parameter to the
    DELETE handler for pit.

rwiker added 2 commits October 4, 2023 14:33
- SumoClient .get() and .delete() no longer decodes the response;
instead, the response object is returned.
- Instead of keyword parameters, SumoClient .get() takes the parameter
'params', which is a dict.
- The parameter 'params' is also added to a few other methods. In
particular, it is now possible to pass a '$pit' parameter to the
DELETE handler for 'pit'.
@rwiker rwiker requested a review from a team October 4, 2023 12:44
Copy link
Contributor

@equinor-ruaj equinor-ruaj left a comment

Choose a reason for hiding this comment

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

lgtm

@rwiker rwiker merged commit 17ed752 into main Oct 5, 2023
4 checks passed
@rwiker rwiker deleted the breaking-changes branch October 5, 2023 07:31
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.

2 participants