Skip to content

Conversation

@yyypearl
Copy link
Contributor

@yyypearl yyypearl commented Jan 23, 2025

연관 이슈

close #130


개요

편지 등록(보내기) 입력 페이지 분할 및 UX 라이팅 수정

✅ 작업 내용

  • 편지 등록(보내기) 입력 페이지 분할
  • 편지 등록(보내기) 프로그래스바 추가
  • 편지 등록(보내기) 이름/편지&사진/미리보기 페이지 UX 라이팅 수정
  • layout 파일로 페이지별 공통 레이아웃 및 NavBar, ProgressBar 적용
  • 편지 등록 letterstore로 라우터명 변경

🖥 구현 결과

20250123_213506.mp4

리뷰 요구사항

  • 공통 layout을 뽑아내면서 보니 letter 라우터 내에 [id] 경로랑 편지 등록 관련 경로가 구성도 다르고 기능 자체도 달라서 letter 보다는 보관의 의미를 가진 store가 더 나을 것 같아서 라우터를 분리했는데 괜찮은가요? store가 전역 저장소에서도 쓰이는 이름이기는 해서 다른 걸로 바꾸는 게 좋은가 싶기도 했습니다!

📝 기타 사항

  • 편지 등록이랑 편지 보내기 페이지가 필드명이나 실명 인증 설명 유무 등의 차이가 있긴 하지만, 화면이 비슷해서 뭔가 따로 분리하면 좋을 거 같기도 한데.. 동작하는 API도 다르고 기능도 다르게 봐야 해서 라우터는 다르게 가는 게 맞는 것 같기도 하더라구요. 파라미터로 분리하는 것도 생각해봤는데 둘이 쓰는 전역 저장소가 달라서 오히려 복잡한가도 싶은데 어떻게 생각하시나요? 혹시 뭔가 좋은 다른 방법이 있을까요?!
  • 반응형은 다시 손 봐야합니다! 반응형은 작업 다 끝나고 할게요😀

@yyypearl yyypearl added 💄 design 사용자 UI 및 CSS 파일 추가 · 수정 ♻️ refactor 코드 리팩토링 labels Jan 23, 2025
@yyypearl yyypearl requested a review from hyo-4 January 23, 2025 12:49
@yyypearl yyypearl self-assigned this Jan 23, 2025
@yyypearl
Copy link
Contributor Author

해결이 잘 안 되네요..! 내일 다시 한번 해보겠습니다

@hyo-4
Copy link
Member

hyo-4 commented Jan 24, 2025

나는 오히려 분리를 하는게 더 좋을 것 같아! 나중에 수정할 부분 생기면 유지보수하기가 더 좋을 듯 싶어서!

@hyo-4
Copy link
Member

hyo-4 commented Jan 24, 2025

store 무난해서 좋은듯!!

@yyypearl
Copy link
Contributor Author

나는 오히려 분리를 하는게 더 좋을 것 같아! 나중에 수정할 부분 생기면 유지보수하기가 더 좋을 듯 싶어서!

오호 이것도 맞는 말이네..! 화면이 계속 바뀔 수 있는 걸 생각 못했어. 좋아 그럼 기존대로 페이지는 따로따로 하고 공통으로 쓰이는 컴포넌트들만 분리하도록 할게😀

@github-actions
Copy link

🎉 Deploy Preview

https://lettering-clrby398v-yyypearls-projects.vercel.app
여기에서 배포 결과 확인하기

@hyo-4 hyo-4 merged commit a222352 into develop Jan 27, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💄 design 사용자 UI 및 CSS 파일 추가 · 수정 ♻️ refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Design] 편지 등록(보내기) 입력 페이지 분할 및 UX 라이팅 수정

3 participants