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

[FIX] 로그아웃 오류 해결 #110

Merged
merged 9 commits into from
Feb 10, 2024
Merged

[FIX] 로그아웃 오류 해결 #110

merged 9 commits into from
Feb 10, 2024

Conversation

Mnseo
Copy link
Owner

@Mnseo Mnseo commented Feb 9, 2024

Issue number and Link

이슈 번호 : #109

Summary

  • 로그아웃 문제 해결
  • 엔드포인트 url 및 헤더 추가 로직 변경
  • 카메라 권한 설정 로직 변경
  • 버튼 event 남은 시간 초기화 emit(1분당) 설정

@eshc123 @arinming
카메라 권한 설정 로직 관련해서 의견을 받습니다
원래 생각한것은 CameraPreview 화면에 들어왔을때 LaunchEffect로 권한 허용 시스템 바텀시트를 띄우는건데 아무리 돌려봐도 안돼서 다음 두가지로 설정했습니다

처음 권한 설정 로직

처음 앱 설치시 카메라 권한 허용 X -> 간단히 만든 화면에서 버튼을 누르면 카메라 시스템 권한 바텀시트가 올라옴 -> 허용시 카메라 보임 / 허용 X시 간단히 만든 화면 그대로

변경 로직 (iOS를 참고했습니다)

처음 앱 설치시 카메라 권한 허용 X -> 출석하기 누를 시 앱 설정으로 이동 -> 권한 허용으로 설정해놔도, 허용 하지 않아도 설정에서 앱으로 돌아오면 빈 화면이 남아있음(이걸 어떻게 처리해야할지 모르겠네요,, 하고싶은건 돌아왔을때의 값을 받아서 허용이면 Camera Preview를 보여주는건데) -> 다시 출석하기를 누르면 허용이면 카메라가 보임

PR Type

  • Feature
  • Bugfix
  • Code Style Update
  • Refactoring
  • Documentation content change
  • Other

Other Information

@Mnseo Mnseo added 🔥 bug Something isn't working 민서 labels Feb 9, 2024
@Mnseo Mnseo requested review from eshc123 and arinming February 9, 2024 18:07
@Mnseo Mnseo self-assigned this Feb 9, 2024
@Mnseo
Copy link
Owner Author

Mnseo commented Feb 10, 2024

@eshc123 @arinming 브랜치 pull 받아서 확인 후 추가 작업까지 완료했습니다 확인해주세요✔️

Copy link
Collaborator

@eshc123 eshc123 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@eshc123 eshc123 merged commit fbc6abb into master Feb 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
민서 🔥 bug Something isn't working 📍 refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants