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

비회원 조회 가능하도록 태그, 카테고리, 템플릿 조회 시 MemberDto 제거 #532

Merged
merged 11 commits into from
Aug 22, 2024

Conversation

jminkkk
Copy link
Contributor

@jminkkk jminkkk commented Aug 21, 2024

⚡️ 관련 이슈

close #495

📍주요 변경 사항

  1. 비회원 조회 가능하도록 태그, 카테고리, 템플릿 조회 시 MemberDto 제거
  2. 권한 확인 로직 및 관련 테스트 제거

🎸기타

+) FakeRepository 사용으로 인해 테스트 거짓 양성이 계속해서 발생하고 있습니다. 이에 대해 논의가 필요할 것 같아요.

@jminkkk jminkkk added feature 기능 추가 zap 리뷰 우선순위가 높은 사항 BE 백엔드 labels Aug 21, 2024
@jminkkk jminkkk added this to the 4차 스프린트 🐔 milestone Aug 21, 2024
@jminkkk jminkkk self-assigned this Aug 21, 2024
@jminkkk jminkkk changed the base branch from feat/495-findAll to dev/be August 21, 2024 13:31
kyum-q
kyum-q previously approved these changes Aug 22, 2024
@HoeSeong123 HoeSeong123 merged commit 359b338 into dev/be Aug 22, 2024
5 checks passed
@HoeSeong123 HoeSeong123 deleted the feat/495-none-member branch August 22, 2024 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 feature 기능 추가 zap 리뷰 우선순위가 높은 사항
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

5 participants