-
Notifications
You must be signed in to change notification settings - Fork 2
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
9번 이슈까지 작업한 내용 main 브랜치로 병합 #15
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Chore: dlehdwo -> 2024-ITEC0401
Redirect URI로 인가코드 받아오기 기능 구현
createdAt, updatedAt 을 기록하기 위한 JPA Auditing 기능 설정
Color, UserColor, Style, UserStyle, User, Coordination Entity 작성 완료. Clothing, CoordinationClothing Entity 및 연관관계 작성해야함
Clothing, CoordinationClothing 엔티티 작성 연관관계 모두 매핑 완료.
마이페이지 열람 API 초안 구현 완료
Feat/entity : Entity 뼈대 작성, 연관관계 매핑, User domain의 마이페이지 열람 기능 구현
회원 가입, 이메일 중복 확인, 로그인 API 개발 완료 구글 소셜 로그인에서 자체 로그인으로 변경되었음
Member 패키지 내용을 User 패키지로 옮김 UserRepository 메소드 생성
회원 기능 API Swagger 작성
UserService interface를 UserColorServiceImpl이 구현하도록 했음.
UserStyleService interface를 UserStyleServiceImpl이 구현하도록 했음.
StyleService interface를 StyleServiceImpl이 구현하도록 했음.
accessToken 으로 user 를 잘 찾아오는지 확인하는 작업 필요함
앞서 Service class 를 Service interface와 ServiceImpl class로 분리하는 과정에서 @OverRide 어노테이션을 추가하는 것을 까먹었었음. 수정완료
Feat: "Color, Style 도메인 관련 작업 & UserProfile 수정 API 작성"
user의 id 값은 반환해줄 필요 없다고 판단하였음. isvalid 필드를 만들어 해당 DTO가 유효한지 여부를 기록하려했으나, 유효하지 않은 경우 빈 내용의 DTO를 반환하도록 결정함.
getUserProfile, updateUserProfile 함수의 상단에 accessToken을 이용해 권한을 확인하는 로직을 추가하였음. flow: accessToken - email 디코딩 - user 찾음 (null일 시 오류 리턴) - 유저 정보 반환 or 유저 정보 업데이트
마이페이지 열람, 마이페이지 수정 API에 대한 스웨거 작성
401 권한 없음을 반환해야하는 경우 HttpStatus.UNAUTHORIZED 를 반환하도록 수정. getUserProfile 함수 상단에 인자로 받아온 accessToken이 토큰이 맞는지 확인하는 로직 추가
Authentication 클래스를 이용해 앞 단에서 인가 여부를 판단할 수 있게 함. 간단하게 getName 으로 이메일을 가져올 수 있음.
…eQuery 적용 삭제 함수는 Transactional하게 이루어져야함. @Trancsational @Modifying @query 태그를 UserColorRepository, UserStyleRepository에 작성함
RDS mysql 서버를 사용 할 예정 따라서 github actions을 이용하여 백엔드 서버 자동배포를 하고싶다. application.yml을 로컬 mysql과 rds 서버를 사용할 때 마다 바꾸며 배포하기에는 불편함이 있음 application.yml 설정 파일을 환경변수 사용하는 방식으로 사용하여 배포할 때 수정없이 편하게 배포할 수 있음
Env: CI/CD 자동배포 application.yml 설정
env: application.yml 정리, github에 환경 변수로 해싱 키 등록 완료
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업한 이슈 번호
#1
#3
#8
#9