Skip to content

fix : 인증 권한 제거 - #143

Merged
MINO1020 merged 1 commit into
developfrom
fix-ketword-api-인증-권한-제거
Mar 1, 2026

Hidden character warning

The head ref may contain hidden characters: "fix-ketword-api-\uc778\uc99d-\uad8c\ud55c-\uc81c\uac70"
Merged

fix : 인증 권한 제거#143
MINO1020 merged 1 commit into
developfrom
fix-ketword-api-인증-권한-제거

Conversation

@MINO1020

@MINO1020 MINO1020 commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

#️⃣연관된 이슈

close #142

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 작업 내용 1
  • 작업 내용 2

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능
    • 키워드 댓글 목록 조회에 페이지네이션 기능 추가
    • 페이지 및 크기 파라미터로 댓글 목록 조회 가능 (기본값: 페이지 0, 크기 3)
    • 더 효율적인 콘텐츠 로딩으로 사용자 경험 개선

@MINO1020 MINO1020 self-assigned this Mar 1, 2026
@MINO1020 MINO1020 linked an issue Mar 1, 2026 that may be closed by this pull request
@MINO1020
MINO1020 merged commit f90cff0 into develop Mar 1, 2026
1 of 2 checks passed
@coderabbitai

coderabbitai Bot commented Mar 1, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7872ce4 and 0fb6c1d.

📒 Files selected for processing (2)
  • src/main/java/teamficial/teamficial_be/domain/keyword/controller/TeamficialLogController.java
  • src/main/java/teamficial/teamficial_be/domain/keyword/service/TeamficialLogService.java

Walkthrough

키워드 API 엔드포인트에서 인증 필수 요구사항을 제거합니다. 컨트롤러의 getKeywordCommentList 메서드에서 AuthDetails 파라미터를 제거하고, 서비스 레이어의 해당 메서드에서 User 파라미터를 제거하여 인증 기능을 탈락시켰습니다.

Changes

Cohort / File(s) Summary
인증 파라미터 제거
src/main/java/teamficial/teamficial_be/domain/keyword/controller/TeamficialLogController.java, src/main/java/teamficial/teamficial_be/domain/keyword/service/TeamficialLogService.java
getKeywordCommentList 메서드에서 AuthDetails/User 파라미터를 제거하고, 페이지네이션 파라미터(page, size)를 @RequestParam으로 추가. 인증 없이 키워드 댓글 목록을 조회할 수 있도록 변경.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-ketword-api-인증-권한-제거

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

[fix] ketword api 인증 권한 제거

1 participant