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

fix : pagenation key 변경 #830

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

jasper200207
Copy link
Collaborator

작업 요약

프로필 작성글 버그픽스

작업 상세 설명

작성글조회 페이지키에 유저아이디를 넣었었는데 표에 빼먹었습니다. 추가후 정상작동 확인했습니다.

리뷰 요구사항

1분

작성글조회 페이지키에 유저아이디를 넣었었는데 표에 빼먹었습니다. 추가후 정상작동 확인했습니다.
@jasper200207 jasper200207 added the Bug 버그 label Oct 17, 2023
@jasper200207 jasper200207 self-assigned this Oct 17, 2023
@publdaze
Copy link
Contributor

publdaze commented Oct 17, 2023

페이지 키에 유저 아이디 넣은 이유가 궁금합니다!
image
url 앞부분에 이미 유저 아이디 구분하고 있어서 페이지 키에도 따로 구분이 필요한가 해서요!

@jasper200207
Copy link
Collaborator Author

페이지 키에 유저 아이디 넣은 이유가 궁금합니다! image url 앞부분에 이미 유저 아이디 구분하고 있어서 페이지 키에도 따로 구분이 필요한가 해서요!

다른 유저 프로필에도 작성글 탭이 있다보니 유저별로 페이지네이션이 유지되지 않고 분리되야 할듯 하여 아이디에 넣어주었습니다. 다른 유저 프로필로 가면 페이지네이션이 초기화 될듯하긴 하지만 일단 있으면 좋지 않을까 싶어서 넣었습니다.

Copy link
Contributor

@publdaze publdaze left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@jasper200207 jasper200207 merged commit 9225a03 into develop Oct 17, 2023
2 checks passed
@jasper200207 jasper200207 deleted the feature/프로필_작성글_버그픽스 branch October 17, 2023 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 버그
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants