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-198 마이페이지 ui 적용 #73

Merged
merged 14 commits into from
Dec 15, 2022
Merged

Conversation

junhaesung
Copy link
Member

@junhaesung junhaesung commented Dec 13, 2022

💁‍♂️ 변경 내용

AS-IS

  • 마이페이지 비어있음

TO-BE

  • 마이페이지 UI 추가
    • 닉네임 수정 다이얼로그
    • versionName 보여주기
  • 습관보관함 UI 일부 추가
  • 버전관리 필요없는 파일 제외 (.idea/misc.xml, .idea//compiler.xml)

📢 전달사항

  • 너무 길어져서 중간에 PR 올립니다
  • 닉네임 수정 다이얼로그 띄웠을 때, 키보드 올라오는거 아직 해결 못했습니다
  • Snackbar 레이아웃 관련 코드에서 lint error 발생하는 부분 있는데, 이것도 아직 해결 못했습니다.

마이페이지

image

닉네임 수정 (enabled)

image

닉네임 수정 (disabled)

image

습관 보관함 (작업중)

image

@junhaesung junhaesung requested a review from a team as a code owner December 13, 2022 22:38
@junhaesung junhaesung self-assigned this Dec 13, 2022
@junhaesung junhaesung added the enhancement New feature or request label Dec 13, 2022
Copy link
Contributor

@kimhyeing kimhyeing left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!!!!

presentation/mypage/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
build.gradle Outdated
ext {
threeDaysApplicationId = "com.depromeet.threedays"
threeDaysAppVersionCode = 2
threeDaysAppVersionName = "1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

저도 시맨틱 버저닝 좋습니다-!

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 주환님 보시고 의견 부탁드려요

Copy link
Member Author

Choose a reason for hiding this comment

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

주환님도 동의하셔서 버저닝 방식 바꿨습니다 (1.0 -> 0.0.1)
QA 끝나고 심사 올릴때 버전올리면서 태그도 붙이면 좋을거같아요

@sonarqubecloud
Copy link

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 13 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@junhaesung junhaesung merged commit 5ef34e4 into develop Dec 15, 2022
@junhaesung junhaesung deleted the feature/DEP-198_mypage-layout branch December 15, 2022 15:25
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