search

woowacourse-teams / 2023-emmsale

18 stars 2 forks source link

Feature/#886 내가 작성한 글 구현 #887

Closed chws0508 closed 7 months ago

chws0508 commented 8 months ago

️⃣ 연관된 이슈

close : #886

📝 작업 내용

내가 작성한 글 화면을 구현하였음

스크린 샷

스크린샷 2023-12-17 오후 10 58 06

스크린샷 2023-12-17 오후 10 56 31

예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)

예상 소요 시간 : 3시간 실제 소요 시간 : 4시간

💬 리뷰어 요구사항 (선택)

지금 행사 제목이 안내려오고 있어서 함께해요 화면의 제목이 빈 상태임

tmdgh1592 commented 8 months ago

대부분 토마스가 리뷰를 많이 해주셨네요! 스타일 관련 리뷰가 많아서 스캇의 생각에 따라 적절히 반영해주시면 좋을 것 같습니다. 저는 따로 리뷰할 부분이 없습니다.

chws0508 commented 8 months ago

RefreshableViewModel을 잘 사용해주셔서 좋았습니다. 보니까 작성한 글이 서버에서 전송해준 순서 그대로 보여주던데, 저는 작성한 댓글을 최신순으로 정렬하고 있습니다. 그리고 날짜는 "yyyy.MM.dd" 형식으로 보여주고 있습니다. 작성한 글 순서 또한 최신순으로 보여주면 어떨까요? 날짜 형식은 스캇이 사용한 형식을 보여줘도 좋다고 생각합니다. 다만 작성한 글과 작성한 댓글의 날짜 형식을 통일하면 좋을 것 같아요. 작성한 댓글의 날짜 형식을 스캇이 사용한 날짜 형식인 "yyyy년 MM월 dd일 X요일"로 바꾸는 게 나을까요?

아뇨 제가 댓글과 같은 형식으로 변경하겠습니다. 그리고 최신순으로 정렬하는 것은 백엔드에서 정렬해서 주는 것이 좋아 보입니다. 일단은 임의로 제가 최신순으로 정렬을 하겠습니다