Merged
Conversation
YuuuuuuYu
approved these changes
Feb 25, 2025
Comment on lines
+29
to
+37
| public static ImageFile createImageFile(int key) { | ||
| return ImageFile.create( | ||
| new ImageFileDto( | ||
| "originalFileName" + key, | ||
| "imageUrl" + key, | ||
| "thumbnailUrl" + key | ||
| ) | ||
| ); | ||
| } |
Member
There was a problem hiding this comment.
filename 뒤에 key 붙는건 이해하는데 Url 뒤에 붙으면 실제로는 ...image.png1 이런식으로 될거 같은데
단순히 더미데이터 용으로 쓰기 위해 넣으신걸까요?
Contributor
Author
There was a problem hiding this comment.
넵 단순 더미데이터 용으로 데이터 확인할 때 알아보기 쉬울라고 추가한거고 딱히 다른 의미는 없습니다!
사실 저게 url인지 생각도 안하고 붙이긴 했어요..ㅎㅎ
Contributor
Author
|
자꾸 개발 하다가 pr 타이밍을 놓쳐서 커져버렸네요.. 담부턴 잘라서 잘 올리겠습니다 ㅠ
이거는 약간 사람마다 다른 것 같아요. 저는 개인적으로 타입 명이 너무 길어지는 경우가 아니면 타입을 명시해주는걸 선호하기는 합니다. 반면에 적극적으로 사용하는 사람도 몇 번 본 것 같아요. 코틀린 하시는 분들은 잘 사용하는 것 같더라구요.. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚀 작업 내용 설명
📢 그 외
투표 변경은 투표하기 기능으로 합쳤습니다.
id로 게시글 조회 기능은 투표 현황 API 분리해야 돼서 수정이 필요해서 추가 구현 필요한 상황이고, docs에 테스트용 access token 추가하고 API 이름 옆에 Http Method 추가했습니다.
📌 관련 이슈
#1 #2 #28