Skip to content

refactor: 사장님 로그인 수정#114

Open
dongcarry96 wants to merge 2 commits intodevfrom
refactor/112-store-login
Open

refactor: 사장님 로그인 수정#114
dongcarry96 wants to merge 2 commits intodevfrom
refactor/112-store-login

Conversation

@dongcarry96
Copy link
Contributor

@dongcarry96 dongcarry96 commented Sep 25, 2025

resolved : #112

체크 리스트

  • 어플리케이션이 정상적으로 실행되나요?
  • Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
  • Push 되지 않은 내역은 없나요?

구현 설명

  • 기존에는 로그인할때 토큰 정보와 fcm만 데이터를 넘겼는데 회원 관련 다른 정보도 넘기도록 수정 했습니다.

요구 사항과 구현 내용

리뷰 포인트 & 궁금한 점

수정전
image
수정후
image

📌 코드리뷰 질문 그라운드룰


✅ 코드에 대한 맥락 없이 리뷰어의 주관적인 의견을 묻거나, 단순히 현업에서는 어떻게 하는지를 묻는 질문은 코드 리뷰 본문에는 어울리지 않아요.
코드 리뷰는 변경된 코드의 목적, 의도, 품질을 중심으로 논의하는 자리예요.


✅ 코드와 관련된 질문이 있다면 PR 본문에 적기보다는, 해당 코드를 선택해 코멘트를 남겨주세요.

@dongcarry96 dongcarry96 self-assigned this Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 사장님 로그인

2 participants