[Feat/ IPLC-13] 댓글 아이템 UI 작업 #188
Open
Conversation
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
MinseoSONG
approved these changes
Jan 6, 2026
Member
MinseoSONG
left a comment
MinseoSONG
left a comment
There was a problem hiding this comment.
완전 짱많이 수고햇슨.
너무 깔끔해서 보면서 기분이 좋앗습니디.
크게 수정할 것들없어보여서 미리 어푸는 찍어놧으니 코리 확인해주고 타 파트랑 논의 다 하면
안드 톡방에 어케 논의됏는지까지 말해주세용 ~
Comment on lines
20
to
21
| // 아래는 필요하지만 문서에 없는것들 | ||
| val likeCount: Int |
Member
There was a problem hiding this comment.
[3]: 문서에 없는 것들이라는게 api 명세서 말하는건가? 지금 반영되어잇어용 ~
Comment on lines
+11
to
+14
| val userJob: String, // 사용자 직무 정보(현재 123순위 기능 구현 안해서, 그냥 직무 정보중 첫번째로 가져옴) | ||
| val state: CertAcquireStateType, // 취득 예정인지, 취득인지 | ||
| val createdTime: String, // 생성일자 | ||
| val lastModifiedTime: String, // 수정일자 |
Member
There was a problem hiding this comment.
[3]: 카톡에도 말했지만 아카이빙을 위해 여기도 남깁니디
- 사용자 희망 직무 정보: 경우(1개인 사람, 2개인 사람, 3개인 사람 - 모두 보여줄건지, 1순위만 보여줄건지)에 따라 어떻게 구현해야할지 -> 기획과 논의 필요
- 피그마 상 수정 기능이 없음. 서버 api에는 수정일자라는게 존재. -> 기획, 서버와 논의 필요
Collaborator
Author
There was a problem hiding this comment.
문의한 결과
- 1순위만 보여주도록
- 서버측에서 추후 들어갈수도 있다고 판단해서 일단 넣어놓은것. 사용 안해도됨
Member
There was a problem hiding this comment.
[3]: 얘를 댓글 컴포넌트 제외 다른 곳에서 사용하는 곳이 있나?
확장성 생각하면 여기 있는게 맞는데, 네이밍부터 commentdeletebutton인 것을 보면 댓글 컴포넌트에서만 사용이 될 것 같아서
댓글 컴포넌트 파일 내부에 private으로 구현해버리는건 어떨지 ~~ 건의해봅니디
| ) | ||
|
|
||
| Text( | ||
| text = "(${commentData.userMajor}, ${commentData.userJob})", |
Comment on lines
92
to
94
| CommentDeleteButton { | ||
| // TODO delete comment | ||
| } |
Member
There was a problem hiding this comment.
[3]: 이거 지금 pr에서 삭제 다이얼로그 연결까지 염두해서 위에 인자 추가해가지고 구현해두면 나중에 더 편할 것 같아욤
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Work Description ✏️
Screenshot 📸
2026-01-05.6.39.02.mov
Uncompleted Tasks 😅
To Reviewers 📢