Skip to content

[정우진] sprint 10 #8

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.

Sign up for GitHub

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

Jump to bottom
Open
jwj3171 wants to merge 11 commits into
base: express-정우진
Choose a base branch
from
Open

[정우진] sprint 10 #8

jwj3171 wants to merge 11 commits into from

Conversation

@jwj3171
Copy link
Collaborator

@jwj3171 jwj3171 commented Jul 27, 2025 •
edited
Loading

요구사항

기본

  • User CRUD
  • Article (자유게시판 게시글) CRUD
  • auth 토큰기반인증
  • commentArticle (자유게시판 게시글-댓글) CRUD
  • commentArticle 좋아요 / 좋아요 취소 토글

TODO

  • refreshToken활용 accessToken 자동갱신
  • multer활용하여 이미지 올리기 (article img, user img)
  • render 연결
  • Article, commentArticle Delete옵션 관련 로직

주요 변경사항

  • article, commentArticle repository, service layer 리팩토링(가공 로직 service 레이어로 이동)

스크린샷

image

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@jwj3171 jwj3171 requested a review from rl0425 July 27, 2025 12:15
@jwj3171 jwj3171 self-assigned this Jul 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rl0425 rl0425 Awaiting requested review from rl0425

At least 1 approving review is required to merge this pull request.

Assignees

@jwj3171 jwj3171

Labels

매운맛🔥 최종 제출

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jwj3171