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

Refactor/http api #55

Closed
wants to merge 5 commits into from
Closed

Refactor/http api #55

wants to merge 5 commits into from

Conversation

zrgt
Copy link

@zrgt zrgt commented Aug 27, 2024

  • Refactored handling of get_response_type():
    To remove get_response_type() in the beginning of all endpoint funcs I have integrate it in handle_request() and added an arg for response type in all endpoint funcs.
  • Refactored _get_submodel_or_nested_submodel_element():
    It has only url_args now as an argument and returns the se or a submodel.
    Before it had a submodel and id_shorts in arguments, so in all funcs which used it a submodel and id_short_path were explored with separate funcs.

Depends on #54

To remove get_response_type() in the
beginning of all endpoint funcs I have
integrate it in handle_request()
and added an arg for response type
in all endpoint funcs.
@s-heppner s-heppner deleted the branch feature/http_api September 3, 2024 10:04
@s-heppner s-heppner closed this Sep 3, 2024
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