Skip to content

Conversation

@kss761036
Copy link
Owner

@kss761036 kss761036 commented Apr 7, 2025

❓이슈

변경 내용

  • 에피그램 작성용 useCreateEpigram 훅 생성 및 적용
  • 에피그램 수정용 useUpdateEpigram 훅 생성 및 적용
  • mutation 관련 로직 분리
  • 중복된 reset() 호출 제거 -> 실제 폼 리셋은 EpigramForm 내부에서만 수행

변경 이유

  • mutation 관련 로직을 각 페이지에서 분리하여 가독성 향상
  • 재사용 가능한 커스텀 훅 구조로 정리
  • 불필요한 중복 리셋 호출 제거

📝 Description

🌀 PR Type

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항 (오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR

  • Branch Convention 확인

    feat/* 기능 구현, fix/* 버그 수정, refactor/* 개선

  • Base Branch 확인
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

@kss761036 kss761036 requested a review from SeokChan-Lee April 7, 2025 08:34
@kss761036 kss761036 linked an issue Apr 7, 2025 that may be closed by this pull request
2 tasks
@kss761036 kss761036 self-assigned this Apr 7, 2025
@vercel
Copy link

vercel bot commented Apr 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
epigram-gilv ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 7, 2025 8:34am

@kss761036 kss761036 added Priority: Medium 🔖 우선순위 보통 Type: Refactor ♻️ 리팩토링 labels Apr 7, 2025
@github-actions
Copy link

github-actions bot commented Apr 7, 2025

Copy link
Collaborator

@SeokChan-Lee SeokChan-Lee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍👍👍

@kss761036 kss761036 merged commit 698a3e9 into main Apr 7, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Medium 🔖 우선순위 보통 Type: Refactor ♻️ 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor: 에피그램 작성페이지 코드 스타일 수정

3 participants