Skip to content

[Feat/#23] - Test Acount API#24

Merged
ttttkii913 merged 1 commit into
developfrom
feature/login-test-account
May 13, 2026
Merged

[Feat/#23] - Test Acount API#24
ttttkii913 merged 1 commit into
developfrom
feature/login-test-account

Conversation

@ttttkii913
Copy link
Copy Markdown
Contributor

@ttttkii913 ttttkii913 commented May 12, 2026

💻 관련 이슈

관련된 이슈 번호를 적어주세요.

✨ 작업 내용

이번 PR에서 작업한 내용을 간단하게 적어주세요.

  • 카카오 로그인 연결 전 테스트 계정으로 액세스 토큰 발급용 API

📄 상세 내용

작업의 상세 설명, 고려한 부분, 코드 구조 등에 대해 설명해 주세요.

  • 데베에 테스트 계정을 넣는 sql문을 통해 테스트 사용자 생성
  • 이후 해당 api 호출만으로 액세스 토큰 발급 => 로그인 연결되면 해당 api 삭제할 예정

📷 스크린샷

관련된 스크린샷이 있다면 첨부해 주세요.

image

✅ 체크리스트

PR을 보내기 전에 아래 항목들을 확인해 주세요.

  • 코드 컨벤션 준수
  • 빌드 성공
  • 로직 자체 테스트 완료
  • 불필요한 파일/코드 제거(개행 정리)
  • 이슈 번호 연결 확인
  • EOL(End Of Line) 확인

❕리뷰 요구 사항

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

  • 이건 수정한 게 별로 없어서 확인하시면 바로 머지하겠습니다 !

@ttttkii913 ttttkii913 self-assigned this May 12, 2026
@ttttkii913 ttttkii913 added the ✨feature 구현, 개선 사항 관련 부분 label May 12, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a test account login feature by adding a /test-account endpoint, a service method to generate tokens for a specific test user, and the necessary test data in the database. Feedback focuses on securing the test endpoint using environment profiles to prevent production exposure and refactoring the implementation to use existing login services for consistency in token handling. Additionally, it is recommended to use constants for hardcoded values and clean up redundant imports.

@ttttkii913 ttttkii913 requested a review from goodispotato May 12, 2026 11:00
@goodispotato
Copy link
Copy Markdown
Contributor

넵 확인했습니다! 머지하셔도 될 것 같아용 수고하셨습니다!!~

@ttttkii913 ttttkii913 merged commit bf21f1b into develop May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨feature 구현, 개선 사항 관련 부분

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants