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.

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

번쩍 개설하기 UI 작업 #978

Merged
merged 6 commits into from
Jan 20, 2025
Merged

번쩍 개설하기 UI 작업 #978

merged 6 commits into from
Jan 20, 2025

Conversation

borimong
Copy link
Contributor

@borimong borimong commented Jan 16, 2025

🚩 관련 이슈

📋 작업 내용

  • 모임 개설하기 페이지의 것을 많이 응용하였어요.
  • 리팩토링할 부분이 많습니다! 여행 이슈로 조금 급하게 작업한 감이 있네요..ㅎㅎ
  • dateError 쪽, error 처리 방식을 기존과 바꾼 부분이 있어서 바꿔줘야 하는데 아직 못 했습니다!
  • 무엇보다도 Presentation 컴포넌트, 아무리 form 이라지만 컴포넌트 분리가 하나도 안 되어 있어요. 컴포넌트 분리가 필요해보입니다!
  • union type 의 state 관리 전략을 선택해서, 명령적으로 코딩하였어요~!

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지
  • 어떤 위험이나 우려가 발견되었는지
  • 어떤 부분에 리뷰어가 집중해야 하는지
  • 개발하면서 어떤 점이 궁금했는지

📸 스크린샷

Copy link

height bot commented Jan 16, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@borimong borimong merged commit e86ffa5 into develop Jan 20, 2025
2 checks passed
@borimong borimong deleted the feat/#977 branch January 20, 2025 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

번쩍 개설 UI 작업
1 participant