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

DEP-163 기기 알림 안내 바텀시트 ui 구현 #50

Merged
merged 3 commits into from
Nov 30, 2022

Conversation

juhwankim-dev
Copy link
Member

💁‍♂️ 변경 내용

KakaoTalk_20221130_021209888

이곧내 (이미지가 곧 내용이라는 뜻)
특이사항으로는 저 중간에 회색 박스 부분은 png로 넣어 달라고 하셔서 png로 넣었습니다.
아마 저기에서 사용하는 폰트가 다른 폰트라 그냥 png로 박기로 하셨나봅니다.
svg로 해도 되는지 여쭤봤고 확정 나면 svg로 바꿔서 머지하고
png로 확정나면 해상도 별 이미지 추가해서 머지하겠습니다. (현재는 xxxhpdi만 있는 상태)

@juhwankim-dev juhwankim-dev added the enhancement New feature or request label Nov 29, 2022
@juhwankim-dev juhwankim-dev requested a review from a team as a code owner November 29, 2022 17:15
@juhwankim-dev juhwankim-dev self-assigned this Nov 29, 2022
Copy link
Member

@junhaesung junhaesung left a comment

Choose a reason for hiding this comment

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

오타 하나만 수정하면 될거같습니당!

@@ -1,3 +1,7 @@
<resources>
<string name="create_habit">습관 만들기</string>
<string name="noti_guide_title">기기 알림을 켜주세요!</string>
<string name="noti_guide_sub_title">알림을 켜면 짝심사일의\n모든 알림을 받아볼 수 있어요.</string>
Copy link
Member

Choose a reason for hiding this comment

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

짝심일의 :D

Copy link
Member Author

Choose a reason for hiding this comment

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

오메 사일로 만들어버렸네요 ㅋㅋㅋ 감사합니다

@juhwankim-dev juhwankim-dev merged commit 711e26c into develop Nov 30, 2022
@sonarcloud
Copy link

sonarcloud bot commented Nov 30, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@juhwankim-dev juhwankim-dev deleted the feature/DEP-163_noti_on_bottom_sheet branch December 10, 2022 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants