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

issue#82 게시글 리팩토링 #83

Merged
merged 18 commits into from
Jan 10, 2025
Merged

Conversation

ppochaco
Copy link
Member

@ppochaco ppochaco commented Jan 10, 2025

📝 상세 내용

  • 활동 게시글과 공지사항 게시글의 API 변경으로 리팩토링 진행
  • 게시글 삭제 시 확인 모달창 보이게 추가
  • shadcn의 캘린더 UI 버전에 오류가 있어 버전 업데이트 후 타입 에러를 직접 수정했습니다..
  • 게시글 작성 시 로컬 이미지 추가하기는 따로 이슈를 만들어서 진행할 예정입니당

#️⃣ 이슈 번호

🔗 참고 자료

📷 스크린샷(선택)

@ppochaco ppochaco requested review from Dobbymin and joojjang January 10, 2025 13:42
@ppochaco ppochaco self-assigned this Jan 10, 2025
@ppochaco ppochaco linked an issue Jan 10, 2025 that may be closed by this pull request
9 tasks
Copy link
Collaborator

@Dobbymin Dobbymin left a comment

Choose a reason for hiding this comment

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

저희가 props 부분에 해당하는 타입은 interface 쓰기로 한것 같은데 반영이 안된것 같네요 ㅠㅠ
캘린더 관련 라이브러리 버전이 바뀌면서 문법이 좀 많이 변경된것 같네요. 고생많으셨습니다.

Comment on lines +35 to +38

# api docs
/src/models
Copy link
Collaborator

Choose a reason for hiding this comment

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

해당 파일들에 eslint 관련 disable 속성이 많이 적용되어있는것 같은데, eslintignore 파일도 생성해서 무시 파일에 추가하는건 어떨까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

해당 파일은 저희가 직접 작성하는 게 아니라, 라이브러리에서 작성해주는 파일이니까 따로 eslintignore에 파일을 추가할 필요는 없을 거 같아요..!

@ppochaco
Copy link
Member Author

저희가 props 부분에 해당하는 타입은 interface 쓰기로 한것 같은데 반영이 안된것 같네요 ㅠㅠ 캘린더 관련 라이브러리 버전이 바뀌면서 문법이 좀 많이 변경된것 같네요. 고생많으셨습니다.

예전에 작성된 코드라서 type으로 사용한 부분이 많네요 ㅎㅎ.. 아래 로직만 변경하느라 놓친 것 같습니다ㅜㅜ 담부턴 주의하겠습니당

@ppochaco
Copy link
Member Author

머지머지

@ppochaco ppochaco merged commit c0ead35 into develop Jan 10, 2025
1 check passed
@ppochaco ppochaco deleted the feature/#82-refactor-post branch January 10, 2025 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

게시글 리팩토링
2 participants