Skip to content

Restructure functions#16

Merged
morukele merged 3 commits intomainfrom
Restructure-Functions
Sep 21, 2023
Merged

Restructure functions#16
morukele merged 3 commits intomainfrom
Restructure-Functions

Conversation

@morukele
Copy link
Owner

In this PR, the goal is to restructure the architecture of the API client. The following actions were taken:

  • The endpoint body and response definitions were moved to files matching the endpoint name.
  • All functions were changed to take &str instead of String. The goal for this is to reduce the instances where the client takes ownership of user-inputted data.

This will be the default way of implementation moving forward.

@morukele morukele assigned morukele and unassigned morukele Sep 21, 2023
@morukele morukele added the enhancement New feature or request label Sep 21, 2023
@morukele morukele self-assigned this Sep 21, 2023
@morukele morukele merged commit c3041fc into main Sep 21, 2023
@morukele morukele deleted the Restructure-Functions branch September 21, 2023 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant