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.

Sign up for GitHub

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

Jump to bottom

style: uuid 변수명 suffix 통일 #229

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

style: uuid 변수명 suffix 통일 #229

wants to merge 2 commits into from

Conversation

letskuku
Copy link
Member

📄구현 내용

📝기타 알림사항

  • 어드민 기능 중 post 아이디로 조회/삭제하는 api들도 uuid로 바꿔둬야할 것 같아 따로 이슈 열겠습니다!

@letskuku letskuku added the style label Jul 22, 2024
@letskuku letskuku self-assigned this Jul 22, 2024
aeyongdodam
aeyongdodam approved these changes Jul 22, 2024
View reviewed changes
Copy link
Contributor

@aeyongdodam aeyongdodam left a comment

Choose a reason for hiding this comment

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

헉 어드민 중에서도 uuid로 두어야하는 이유는 뭐라고 판단하셨나요?
저번에 pr올렸을 때는 id로 가져오는 방향으로 논의가 나왔던 것 같은데 제가 어떤 걸 놓쳤는지 알고 싶습니다!!

@letskuku
Copy link
Member Author

그때 소현님께서 말씀해주신 이유로 수정했습니다!
다른 어드민 기능들의 경우, 필요한 엔티티 '목록'을 가져오는 api가 있어 어드민 api들만 가지고도 아이디 확인이 가능합니다
하지만 post의 경우, 아이디로 특정 post를 조회하고 삭제하는 api들만 있어 특정 post의 아이디를 아는 방법은 서비스 화면 url로 파악하는 것인데 url에 나타나는 아이디가 uuid라 수정이 필요하다고 판단했습니다

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

@aeyongdodam aeyongdodam aeyongdodam approved these changes

Assignees

@letskuku letskuku

Labels
style
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

style: uuid 변수명 suffix 통일
2 participants
@letskuku @aeyongdodam