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

[UI/YAF-67] 온보딩 접근권한 설정 UI를 구현합니다. #38

Merged
merged 6 commits into from
Jan 15, 2025

Conversation

MoonsuKang
Copy link
Member

@MoonsuKang MoonsuKang commented Jan 15, 2025

Related issue 🛠

closed #33

어떤 변경사항이 있었나요?

  • 🐞 BugFix Something isn't working
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 컨벤션에 맞게 작성했습니다. (필수)
  • merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
  • BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

  • 접근권한 UI 구현
Screen_recording_20250115_150523.webm

Uncompleted Tasks 😅

  • Task1

To Reviewers 📢

우선 UI 구현을 처리했고, 실제 시스템 알림 권한 설정은 새로운 이슈를 통해 진행할 예정입니다!

@MoonsuKang MoonsuKang added 📱 UI 사용자 인터페이스 관련 작업 👊 문수문수문수문수문수 labels Jan 15, 2025
@MoonsuKang MoonsuKang requested a review from DongChyeon January 15, 2025 06:06
@MoonsuKang MoonsuKang self-assigned this Jan 15, 2025
Copy link
Member

@DongChyeon DongChyeon left a comment

Choose a reason for hiding this comment

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

xhdpi 이상만 가져온 이유는 대부분의 기기가 xhdpi 이상 밀도이기 때문인가요?

@MoonsuKang
Copy link
Member Author

@DongChyeon
넵 우선 그렇게 생각해서 넣어두었습니다!
아마 안드로이드 5인가?6인가? 까지 지원하는 디바이스인 경우 mdpi나 hdpi까지 지원하는 걸로 알고있는데 이거 한 10년? 전 디바이스라 minsdk 조건도 못 맞출 것 같아서 우선 이렇게 처리했습니다~

Copy link
Member

@DongChyeon DongChyeon left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@MoonsuKang MoonsuKang merged commit a146918 into develop Jan 15, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👊 문수문수문수문수문수 📱 UI 사용자 인터페이스 관련 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] 온보딩 접근권한 설정 UI를 구현합니다.
2 participants