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

[fix] GET /authorize API Response 수정 #270

Closed
1 task
KWY0218 opened this issue Jun 25, 2024 · 0 comments · Fixed by #271
Closed
1 task

[fix] GET /authorize API Response 수정 #270

KWY0218 opened this issue Jun 25, 2024 · 0 comments · Fixed by #271
Assignees
@KWY0218
Labels
fix 작업 수정

Comments

@KWY0218
Copy link
Contributor

KWY0218 commented Jun 25, 2024

About Issue 🚀

redirect uri를 웹 로컬에 저장함으로써 웹은 로컬 관리까지 수반됐습니다.
플랫폼 인가코드와 함께 리다이렉션 uri를 합쳐서 redirect url 형태로 넘겨주면서 웹의 부담을 줄이기로 결정했습니다.

Progress ✔️

  • api reseponse 형태 수정
@KWY0218 KWY0218 added the fix 작업 수정 label Jun 25, 2024
@KWY0218 KWY0218 self-assigned this Jun 25, 2024
KWY0218 added a commit that referenced this issue Jun 25, 2024
@KWY0218
#270 [fix] response dto 수정
e6685d4 
api 문서 수정으로 redirect url을 반환함에 따라 dto 수정
KWY0218 added a commit that referenced this issue Jun 25, 2024
@KWY0218
#270 [fix] response 메시지 수정
4295dae
KWY0218 added a commit that referenced this issue Jun 25, 2024
@KWY0218
#270 [feat] response data 수정
24e49b3 
redirect url template을 상수로 만들어서
redirect uri와 platformCode를 redirctUrl 형태로 만든다
KWY0218 added a commit that referenced this issue Jun 25, 2024
@KWY0218
#270 [test] /authorize 테스트 코드 수정
fa92952
@KWY0218 KWY0218 mentioned this issue Jun 25, 2024
Merged
@KWY0218 KWY0218 linked a pull request Jun 25, 2024 that will close this issue
#270 [fix] /authorize 명세 수정 #271
Merged
KWY0218 added a commit that referenced this issue Jun 26, 2024
@KWY0218
#270 [chore] template에서 format으로 상수명 수정
0859f5d
github-merge-queue bot pushed a commit that referenced this issue Jun 26, 2024
@KWY0218
Merge pull request #271 from sopt-makers/feat/#270_T-10863
d0cbcad 
#270 [fix] /authorize 명세 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KWY0218 KWY0218

Labels
fix 작업 수정
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#270 [fix] /authorize 명세 수정 sopt-makers/sopt-operation-backend
1 participant
@KWY0218