Skip to content

add: get_nights_of_user endpoint && OpenAPI specs#60

Open
panosfol wants to merge 6 commits intodevelopmentfrom
get-nights-of-user-endpoint
Open

add: get_nights_of_user endpoint && OpenAPI specs#60
panosfol wants to merge 6 commits intodevelopmentfrom
get-nights-of-user-endpoint

Conversation

@panosfol
Copy link
Copy Markdown
Contributor

No description provided.

@panosfol panosfol requested a review from charmitro November 25, 2022 23:38
@panosfol panosfol added enhancement New feature or request database API labels Nov 25, 2022
Copy link
Copy Markdown
Contributor

@charmitro charmitro left a comment

Choose a reason for hiding this comment

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

No need to create a new endpoint. Simply use the nights/ endpoint. Just add a query parameter there and act accordingly.

Comment thread api/src/main.rs Outdated
@panosfol panosfol requested a review from charmitro November 26, 2022 03:20
Comment thread api/src/controllers/nights.rs Outdated
Comment thread api/src/controllers/nights.rs Outdated
@panosfol panosfol requested a review from charmitro November 26, 2022 12:54
Copy link
Copy Markdown
Contributor

@charmitro charmitro left a comment

Choose a reason for hiding this comment

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

Fix all failing tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API database enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants