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 : 온보딩 3단계 키워드 알림 설정 페이지 구현 #155

Closed
wants to merge 55 commits into from

Conversation

huiwoo-jo
Copy link
Contributor

@huiwoo-jo huiwoo-jo commented Feb 12, 2024

📮 관련 이슈

✍️ 구현 내용

  • 온보딩 3단계 키워드 알림 설정 페이지를 구현하였습니다.
  • 온보딩 브랜치가 머지되지 않아 온보딩 2단계 구현 브랜치에서 중복되는 사항이 있습니다.
  • 화면 상단에 화면 설명 부분을 추가하고 텍스트에 부분 색상을 설정하였습니다.
  • 온보딩 단계에서 '알림 설정'이 활성화 됩니다.
  • 키워드 알림설정 스위치 박스를 제작하였습니다.
  • 스위치 박스 활성화에 따라 텍스트와 그림자의 색상이 변화합니다.
  • 좌측의 leftDrawable의 색상은 아직 변화 방법을 찾지 못하였습니다. 의견이 있다면 말씀해주세요.
  • 하단의 '알림 설정 및 시작하기'로 메인 화면으로 이동합니다.

📷 구현 영상

KakaoTalk_20240212_215613794.mp4
image image

✔️ 확인 사항

  • 컨벤션에 맞는 PR 타이틀
  • 관련 이슈 연결
  • PR 관련 정보 연결 (작업자, 라벨, 마일스톤 등)
  • Github Action 통과

@huiwoo-jo huiwoo-jo added 🤗 FEATURE Develop this project 🩵희우 labels Feb 12, 2024
@huiwoo-jo huiwoo-jo added this to the 1차 스프린트 milestone Feb 12, 2024
@huiwoo-jo huiwoo-jo self-assigned this Feb 12, 2024
@hoyahozz hoyahozz changed the base branch from develop to design/onboarding-#2 February 13, 2024 11:00
@hoyahozz
Copy link
Member

이전 PR (#154)과 마찬가지로 베이스 브랜치를 변경하였습니다.

Copy link
Member

@hoyahozz hoyahozz left a comment

Choose a reason for hiding this comment

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

코멘트 확인 부탁드려요!

Copy link
Collaborator

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

settingAlarm 보다는 permission 이라는 네이밍이 더 맞지 않을까 싶은데 어떻게 생각하시나요 ?!
ㄴ 이유로는 1. 패키지명은 현재까지 모두 소문자로, 영단어 하나로만 이루어져있습니다.
2. 추후에 기능 추가 시 다른 권한이 필요할 때 이 부분에서 권한을 받게 될 것 같습니다.

Copy link
Collaborator

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

settingAlarm 보다는 permission 이라는 네이밍이 더 맞지 않을까 싶은데 어떻게 생각하시나요 ?!
ㄴ 이유로는 1. 패키지명은 현재까지 모두 소문자로, 영단어 하나로만 이루어져있습니다.
2. 추후에 기능 추가 시 다른 권한이 필요할 때 이 부분에서 권한을 받게 될 것 같습니다.

Base automatically changed from design/onboarding-#2 to develop February 17, 2024 19:04
huiwoo-jo and others added 25 commits March 12, 2024 20:49
@huiwoo-jo huiwoo-jo closed this Mar 12, 2024
@huiwoo-jo huiwoo-jo deleted the feat/onboarding-#3 branch March 12, 2024 11:55
@huiwoo-jo huiwoo-jo restored the feat/onboarding-#3 branch March 12, 2024 11:59
@huiwoo-jo huiwoo-jo deleted the feat/onboarding-#3 branch March 13, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤗 FEATURE Develop this project 🩵희우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

온보딩 화면 3단계 제작
3 participants