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

마지막 페이지 여부를 반환합니다. #38

Merged
merged 2 commits into from
Feb 3, 2024
Merged

Conversation

litsynp
Copy link
Contributor

@litsynp litsynp commented Feb 3, 2024

마지막 페이지 여부 is_last_page를 반환합니다.

가장 오버헤드가 적은 것으로 보이는, size + 1만큼 가져와 길이 비교하는 방식으로 진행했습니다. 이는 PaginatedView struct의 CalcLastPage 함수로 진행합니다.

추가로 돌봄급구 조건 목록 조회 API에서 nil array가 반환되는 이슈를 수정합니다.

@litsynp litsynp added fix Fix bugs feat New feature labels Feb 3, 2024
@litsynp litsynp requested a review from barabobBOB February 3, 2024 04:31
@litsynp litsynp self-assigned this Feb 3, 2024
@litsynp
Copy link
Contributor Author

litsynp commented Feb 3, 2024

Base automatically changed from feat/req-res to main February 3, 2024 16:31
# Conflicts:
#	cmd/server/handler/breed_handler.go
#	cmd/server/handler/sos_post_handler.go
#	cmd/server/handler/user_handler.go
#	internal/postgres/breed_store.go
#	internal/postgres/sos_post_store.go
#	internal/postgres/user_store.go
Copy link
Member

@barabobBOB barabobBOB left a comment

Choose a reason for hiding this comment

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

확인했습니다 :)

@litsynp
Copy link
Contributor Author

litsynp commented Feb 3, 2024

🙇

@litsynp litsynp merged commit 781c7b2 into main Feb 3, 2024
1 check passed
@litsynp litsynp deleted the feat/last-page branch February 3, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature fix Fix bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants