Skip to content

Conversation

@leebs0521
Copy link
Collaborator

@leebs0521 leebs0521 commented Jan 21, 2025

resolved :

📌 과제 설명

NewCenter, UserCommonAttribute 추가에 따른 모집글 리팩토링

👩‍💻 요구 사항과 구현 내용

  • @userid, @RoleId 테스트를 위한 @MockUser 추가
  • CurrentUser -> RoleId 변경
  • 모집글 도메인에서 이미지 관련 로직 삭제
  • Center 관련 로직 NewCenter 의존

✅ PR 포인트 & 궁금한 점

나중에 시간이 되면 조인말고 객체로 풀겠습니다.

@leebs0521 leebs0521 linked an issue Jan 21, 2025 that may be closed by this pull request
3 tasks
@leebs0521 leebs0521 self-assigned this Jan 21, 2025
@leebs0521 leebs0521 force-pushed the refactor/312-refactor-recruit-board branch from a51b558 to 7f79c0f Compare January 21, 2025 08:23
@sonarqubecloud
Copy link

Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 벌써 거의 해결하신 것 같네요?? 빠르십니다. 근데 이미지는 빠진다고 대략은 들었는데 어떻게 관리되는걸까요? 프론트에서 추가적으로 어떤 글과 관련된 이미지라고 서버에게 요청을 보내는 형태일까요?

Copy link
Collaborator

@ayoung-dev ayoung-dev left a comment

Choose a reason for hiding this comment

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

이미지 링크 빠지는 부분도 잘 확인했습니당
수고하셨습니다~

@leebs0521 leebs0521 merged commit b4c28c1 into main Jan 21, 2025
3 checks passed
@leebs0521 leebs0521 deleted the refactor/312-refactor-recruit-board branch January 21, 2025 08:48
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.

[REFACTOR] 테이블 수정에 따른 모집글 리팩토링

4 participants