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] 구글 OAuth 요청 uri 생성 시 prompt 설정을 제거한다. #520

Merged
merged 1 commit into from
Aug 19, 2022
Merged

[fix] 구글 OAuth 요청 uri 생성 시 prompt 설정을 제거한다. #520

merged 1 commit into from
Aug 19, 2022

Conversation

hyeonic
Copy link
Collaborator

@hyeonic hyeonic commented Aug 18, 2022

  • 🔀 PR 제목의 형식을 잘 작성했나요? e.g. [feat] PR을 등록한다.
  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 💻 git rebase를 사용했나요?
  • 🌈 알록달록한가요?

작업 내용

google OAuth 인증 요청을 위한 uri 생성 시 매 로그인시 마다 동의를 요청하지 않도록 prompt 설정을 제거한다.

스크린샷

주의사항

  • local 및 dev 환경에서 테스트할 때 refresh token을 발급 받지 못할 수 있습니다. 관련하여 이야기 나눈 뒤 해당 PR 반영하겠습니다.
  • 서브모듈이 수정되었습니다. merge가 될 경우 서브모듈을 적절히 최신화 시켜야 합니다.

Closes #518

@hyeonic hyeonic added the backend 백엔드 label Aug 18, 2022
@hyeonic hyeonic self-assigned this Aug 18, 2022
gudonghee2000
gudonghee2000 approved these changes Aug 18, 2022
View reviewed changes
Copy link
Collaborator

@gudonghee2000 gudonghee2000 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

prompt를 제거해서 사용자의 달록 접근이 더 쉬워지겠군요!🚀🚀
apporve하였습니다!

summerlunaa
summerlunaa approved these changes Aug 18, 2022
View reviewed changes
Copy link
Collaborator

@summerlunaa summerlunaa left a comment

Choose a reason for hiding this comment

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

한결 편해지겠군요👍

@hyeonic hyeonic merged commit 85d6729 into woowacourse-teams:develop Aug 19, 2022
@hyeonic hyeonic deleted the feature/518-remove-prompt branch August 19, 2022 00:46
@hyeonic hyeonic changed the title 구글 OAuth 요청 uri 생성 시 prompt 설정을 제거한다. [fix] 구글 OAuth 요청 uri 생성 시 prompt 설정을 제거한다. Aug 19, 2022
@hyeonic hyeonic mentioned this pull request Aug 19, 2022
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@summerlunaa summerlunaa summerlunaa approved these changes

@gudonghee2000 gudonghee2000 gudonghee2000 approved these changes

@devHudi devHudi Awaiting requested review from devHudi

Assignees

@hyeonic hyeonic

Labels
backend 백엔드
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

구글 OAuth 요청 uri 생성 시 prompt 설정을 제거한다.
3 participants
@hyeonic @gudonghee2000 @summerlunaa