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: 비로그인 사용자도 리뷰 목록을 볼 수 있도록 수정 #48

Merged
merged 10 commits into from
Apr 23, 2024

Conversation

70825
Copy link
Member

@70825 70825 commented Apr 21, 2024

Issue

✨ 구현한 기능

  • 비로그인 사용자도 리뷰 목록을 볼 수 있도록 수정했습니다.

📢 논의하고 싶은 내용

  • 테스트 코드는 로그인 / 비로그인으로 나누고, 비로그인 메서드를 오버로딩으로 만든게 끝입니다.
  • 프로덕트 코드에 논의할만한 내용 아래에 리뷰로 남겨둘게요

🎸 기타

⏰ 일정

  • 추정 시간 : 5
  • 걸린 시간 : 6

Copy link

github-actions bot commented Apr 21, 2024

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit f863427. ± Comparison against base commit d3665bd.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@Go-Jaecheol Go-Jaecheol left a comment

Choose a reason for hiding this comment

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

설명이 잘 되어있어서 이해하기 편했네요 👍👍
논의해야 되는 부분에 대해서는 코멘트로 남겼는데, 51대 49 정도의 의견이라 로건 의견이랑 다른 사람들 의견 종합해서 정하면 될 것 같아요

Copy link
Contributor

@wugawuga wugawuga left a comment

Choose a reason for hiding this comment

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

설명 차근차근 되어있어서 바로 파악하기에 너무 좋았습니다.
수고했어요오오~

@70825 70825 merged commit 0824d5b into develop Apr 23, 2024
3 checks passed
@70825 70825 deleted the refactor/issue-47 branch April 23, 2024 03:21
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.

비로그인 사용자도 리뷰 목록을 볼 수 있도록 수정
3 participants