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

[Refactor] 관리자 디렉터리 분리 #69

Merged
merged 8 commits into from
Dec 30, 2024

Conversation

limseohyeon
Copy link
Collaborator

@limseohyeon limseohyeon commented Dec 30, 2024

#️⃣ 연관된 이슈

#68

✅ 체크리스트

  • 🔀 PR 제목의 형식 맞추기 ex) [feat] ㅇㅇ 기능 추가
  • 💡 이슈 등록
  • 🏷️ 라벨 등록
  • 🧹 불필요한 코드 제거
  • 🧪 로컬 테스트 완료
  • 🏗️ 빌드 성공
  • 💯 테스트 통과

📝 작업 내용

관리자 디렉터리 분리 및 중복 파일 제거

  • 관리자 디렉터리 분리

  • Member과 Bookmark의 중복된 dto, mapper 파일 제거

📸 스크린샷 (UI 변경 시 필수)

👀 리뷰어 가이드라인

여기에 작성

@limseohyeon limseohyeon self-assigned this Dec 30, 2024
Copy link

github-actions bot commented Dec 30, 2024

Test Results

30 tests   30 ✅  6s ⏱️
11 suites   0 💤
11 files     0 ❌

Results for commit da87b95.

♻️ This comment has been updated with latest results.

@DongminL
Copy link
Collaborator

image

빌드 경고에 따르면 AdminMemberResponse 클래스의 bookmarkResponses 필드가 매핑이 안 된거 같은데, 혹시 실제 테스트에서는 잘 매핑 되는지 확인 하셨나요?

@limseohyeon
Copy link
Collaborator Author

image

빌드 경고에 따르면 AdminMemberResponse 클래스의 bookmarkResponses 필드가 매핑이 안 된거 같은데, 혹시 실제 테스트에서는 잘 매핑 되는지 확인 하셨나요?

넵 실제 테스트에서 매핑 되는지 확인하고 올리긴 했습니다!!
그런데 말씀 주신 내용 확인하니 빌드 경고 안 뜨게 하는 게 좋을 것 같아 해당 부분 수정해서 다시 올렸어요...!

@DongminL
Copy link
Collaborator

지금 rebase and merge가 안 됩니다.

image

main 브랜치와 merge한 커밋이 문제 같습니다. merge 대신 rebase로 해주시면 될 것 같아요!

@limseohyeon
Copy link
Collaborator Author

넵 알겠습니다...! 제가 지금 밖이라 돌아가서 수정하겠습니다

@limseohyeon limseohyeon force-pushed the refactor/#68-관리자-디렉터리-분리 branch 2 times, most recently from cf0d19d to fbb220a Compare December 30, 2024 15:54
@limseohyeon limseohyeon force-pushed the refactor/#68-관리자-디렉터리-분리 branch from fbb220a to da87b95 Compare December 30, 2024 16:04
@limseohyeon limseohyeon merged commit 75ad1ca into main Dec 30, 2024
3 checks passed
@limseohyeon limseohyeon linked an issue Dec 30, 2024 that may be closed by this pull request
@DongminL DongminL deleted the refactor/#68-관리자-디렉터리-분리 branch December 31, 2024 04:06
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.

관리자 디렉터리 분리
2 participants