Skip to content
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.

Sign up for GitHub

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

Jump to bottom

#23 [REFACTOR] 알람, 미션 API 수정 및 Swagger 어노테이션 추가 #29

Merged
merged 10 commits into from
Aug 23, 2023
Merged

#23 [REFACTOR] 알람, 미션 API 수정 및 Swagger 어노테이션 추가 #29

merged 10 commits into from
Aug 23, 2023

Conversation

05AM
Copy link
Member

@05AM 05AM commented Aug 20, 2023 •
edited
Loading

  • 알람, 미션 API 리팩토링
  • Swagger 어노테이션 추가
  • Swagger 에러 해결
  • health check 추가

05AM added 10 commits August 20, 2023 22:57
@05AM
#27 [FIX] 도메인 이름 오타 수정
b26601c 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM
#23 [FEATURE] health check 추가
645a41d
@05AM
#23 [FIX] sql문 수정 - alarm : isActive -> isActivated
2458fdb 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM
#23 [FIX] sql문 수정 - alarm : icon default ⏰
253afe1 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM
#23 [FIX] sql문 수정 - mission : name -> varchar(20)
50e530e 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM
#23 [REFACTOR] 알람 API 수정 및 Swagger 어노테이션 추가
661550a 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM
#23 [REFACTOR] jwt service 필요없는 메소드 삭제, auth 로그인 값 변경
ca0fed3 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM
#23 [REFACTOR] Mission API 리팩토링 및 Swagger 어노테이션 추가
dd0a5cb 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM
#23 [REFACTOR] Swagger 어노테이션 추가, 요청 server 추가
04565c0 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM
#23 [FIX] swagger 403 error 해결
a2819c2 
# #이슈번호 [커밋타입] 내용

#   [FEAT]      : 새로운 기능 구현
#   [FIX]       : 버그, 오류 해결, 코드 수정
#   [MERGE]     : 머지, 충돌 해결
#   [CHORE]     : 빌드 태스트 업데이트, 패키지 매니저를 설정하는 경우(프로덕션 코드 변경 X)
#   [TEST]      : 테스트 코드, 리펙토링 테스트 코드 추가
#   [DOCS]      : README나 WIKI 등의 문서 개정
#   [REFACTOR]  : 프로덕션 코드 리팩토링
#   [COMMENT]   : 필요한 주석 추가 및 변경
#   [INIT]   : 초기 세팅
#   [RENAME]    : 파일 혹은 폴더명을 수정
#   [REMOVE]    : 파일 혹은 폴더명을 옮기는 작업

# Commit Example!!
# [Feat]#1 - ~~기능 추가!
#
# 설명을 덧붙이고 싶다면 이 예시처럼 위에 commit 메세지와 한 칸 띄고 작성하면 돼용  <바디에 들어감>
@05AM 05AM added the REFACTOR refactored code label Aug 20, 2023
@05AM 05AM self-assigned this Aug 20, 2023
hye-on
hye-on reviewed Aug 21, 2023
View reviewed changes
src/main/java/com/wakeUpTogetUp/togetUp/api/auth/service/AuthService.java
@@ -45,7 +45,7 @@ public LoginRes socialLogin(SocialLoginReq socialLoginReq) {

return LoginRes.builder()
.userId(userId)
.useName("박건우")//socialUserRes.getName())
.useName(socialUserRes.getName()) //socialUserRes.getName())
Copy link
Contributor

Choose a reason for hiding this comment

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

아 이거 놓쳤었네
땡큐

hye-on
hye-on reviewed Aug 21, 2023
View reviewed changes
src/main/java/com/wakeUpTogetUp/togetUp/api/mission/MissionService.java
Comment on lines 47 to 48
.orElseThrow(
() -> new BaseException(Status.INVALID_USER_ID));
Copy link
Contributor

Choose a reason for hiding this comment

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

p5 : USER_NOT_FOUND 에러 추가해놔서 이걸로 바꿔도 좋을듯!

Copy link
Member Author

Choose a reason for hiding this comment

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

오키 반영할게!

hye-on
hye-on reviewed Aug 21, 2023
View reviewed changes
src/main/java/com/wakeUpTogetUp/togetUp/config/OpenApiConfig.java
@@ -21,7 +21,8 @@ public OpenAPI openApi() {

return new OpenAPI()
.addServersItem(new Server().url("http://localhost:9010"))
.addServersItem(new Server().url("https://togeptup.shop"))
.addServersItem(new Server().url("http://togetup.shop"))
.addServersItem(new Server().url("https://togetup.shop"))
Copy link
Contributor

Choose a reason for hiding this comment

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

p4 : 두개 들어간것 같아!

Copy link
Contributor

Choose a reason for hiding this comment

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

아 하나는 http네

Copy link
Member Author

Choose a reason for hiding this comment

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

https 안하려다가 해서 빼먹었네 삭제할게!

hye-on
hye-on reviewed Aug 21, 2023
View reviewed changes
src/main/java/com/wakeUpTogetUp/togetUp/utils/JwtService.java
Comment on lines +100 to +107
private Claims getBody(final String token) {
return Jwts.parserBuilder()
.setSigningKey(getSigningKey())
.build()
.parseClaimsJws(token)
.getBody();
}

Copy link
Contributor

Choose a reason for hiding this comment

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

이 함수는 왜 만든건지 궁금해

Copy link
Member Author

Choose a reason for hiding this comment

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

함수 복잡도를 줄이려고 메소드를 추출했어!

hye-on
hye-on reviewed Aug 21, 2023
View reviewed changes
Copy link
Contributor

@hye-on hye-on left a comment

Choose a reason for hiding this comment

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

고생했어!

@hye-on hye-on merged commit 29a9750 into main Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hye-on hye-on hye-on approved these changes

Assignees

@05AM 05AM

Labels
REFACTOR refactored code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@05AM @hye-on