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.

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

#25 Feature/edit user group #28

Merged
merged 3 commits into from
Aug 20, 2023
Merged

#25 Feature/edit user group #28

merged 3 commits into from
Aug 20, 2023

Conversation

hye-on
Copy link
Contributor

@hye-on hye-on commented Aug 20, 2023

구현

  • fcmToken 업데이트
  • 알림수진값 업데이트
  • custom error의 status 코드를 response status 코드에 반영하게 세팅

api

  • 로그인, fcmToken 업데이트, 알림수진값 업데이트 문서 작성
  • swagger 문서 작성은 보완할 예정입니다.

Copy link
Member

@05AM 05AM left a comment

Choose a reason for hiding this comment

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

스웨거 적용 엄청 빨리했네! 고생했어!!

Comment on lines +24 to +30
protected ResponseEntity<BaseResponse<Status>> handleCustomException(BaseException exception) {
logger.debug("BaseResponse exception occurred: {}", exception.getMessage(), exception);

return new BaseResponse<>(exception.getStatus());
Status status = exception.getStatus();
ResponseEntity<BaseResponse<Status>> responseEntity = ResponseEntity.status(status.getHttpStatus())
.body(new BaseResponse<>(status));
return responseEntity;
Copy link
Member

Choose a reason for hiding this comment

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

혹시 왜 responseEntity로 바꿨는지 알 수 있을까?
나도 바꾸려고 생각했는데 이유를 아직 못 찾아서 실행하진 않았어서!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

우리가커스텀 에러를 내려줄때 data값으로 상태코드를 표시하는데
에러가 나도 상태코드가 200으로 나와서 일치시키려고 했어!

@05AM 05AM merged commit 3d76685 into main Aug 20, 2023
@hye-on hye-on deleted the feature/edit-user-group branch October 15, 2023 01:08
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.

2 participants