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

[FEAT,FIX] : 개발용 컨트롤러 추가 , 애플 로그인 탈퇴 수정 #45

Merged
merged 1 commit into from
Sep 15, 2023

Conversation

hye-on
Copy link
Contributor

@hye-on hye-on commented Sep 15, 2023

작업

  • 개발용 컨트롤러 추가 : user 목록, jwt 발급
    (기존의 코드는 클라이언트에서 토큰을 주어야 jwt를 발급 받을 수 있기 때문에
    user 목록에서 가져온 userId를 가지고 jwt 발급 받을 수 있게 하였습니다.)
  • ec2 환경에서 AuthKey_M992KTZK9V.p8를 읽을 수 있게 수정하였습니다.

Comment on lines 475 to 481
<div class="sect3">
<h4 id="_알람_단일_조회_path_parameters">Path parameters</h4>
<table class="tableblock frame-all grid-all fit-content stretch">
<caption class="title">Table 1. /app/alarm/{alarmId}</caption>
<caption class="title">Table 1. /app/alarms/{alarmId}</caption>
<colgroup>
<col>
<col>
Copy link
Member

Choose a reason for hiding this comment

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

혹시 올리기 전에 build를 해보셨을까요? 테스트 코드에 주석을 쳤는데 왜 지금 이렇게 업데이트 되었는지 의문입니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

빌드해서 올렸습니다. 혹시 몰라 다시 빌드해봤는데 그대로입니다!

Copy link
Member

Choose a reason for hiding this comment

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

넵 알겠습니다! 원인을 찾아보겠습니다!

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.

확인 했습니다!

@05AM 05AM merged commit 75f7754 into main Sep 15, 2023
@hye-on hye-on deleted the feature/edit-apple-login branch September 18, 2023 08:01
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