-
Notifications
You must be signed in to change notification settings - Fork 101
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(member): rename user to member #743
Conversation
chore: api요청 user -> member 키워드 변경
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
로컬에서 회원가입 제외 정상 동작 확인했습니다.
회원가입은 이메일 인증 때문에 로컬에서 확인을 못했는데, dev 배포하면 그 때 체크해 볼게요.
여전히 Administrator, SecurityUtils에 user가 있는데, 해당 부분이 적절한지 검토해 주세요.
username
이나 isUserLoggedIn
때문에 그러신건가요?
사실 심정적으로는 안바꿔도 될 것 같은데 명확한 근거를 언급하기 어렵네요. ㅋㅋㅋㅋ(바꿔야할 큰 이유를 못찾겠어요)
별개로 로컬에서 테스트하다보니 헷갈리는 부분들이 생겼는데 이건 주간회의에서 문의드릴게요.
(클라이언트의 더미 데이터 사용, 브랜치 전략)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
프론트 실행에 어려움이 있어서 백엔드만 테스트해서 문제 없는거 확인했습니다..ㅋㅋ
고생하셨습니다~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
회원과 관련된 모든 기능 훑었습니다. 문제없이 동작하는 것을 확인했습니다.
고생하셨습니다 👏👏👏👏
- 회원/회원 정보
- 🟢 회원가입(DB에서 인증번호 긁어서 입력)
- 🟢 비밀번호 수정 후 재로그인
- 🟢 휴대폰 번호 수정
- 🟢 로그인/로그아웃
- 지원하기
- 🟢 지원 항목 임시 저장
- 🟢 최종 제출
Resolves #737
해결하려는 문제가 무엇인가요?
user
를 예약어로 사용하여 개발에 불편을 초래하고 있습니다.어떻게 해결했나요?
user
대신 한글명인 '회원'에 더 가까운member
로 변경하였습니다.어떤 부분에 집중하여 리뷰해야 할까요?
Administrator
,SecurityUtils
에user
가 있는데, 해당 부분이 적절한지 검토해 주세요.참고 자료
RCA 룰
r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
c: 웬만하면 반영해 주세요. (Comment)
a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)