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

[QA] 운동 뷰 QA 1차 수정 #140

Merged
merged 28 commits into from
Apr 8, 2024
Merged

[QA] 운동 뷰 QA 1차 수정 #140

merged 28 commits into from
Apr 8, 2024

Conversation

hyunjium
Copy link
Collaborator

@hyunjium hyunjium commented Apr 3, 2024

📌 관련 이슈

📷 screenshot

📝 Work Desciption

  • 히스토리랑 오늘 미션 둘 다 없을 때 빨간색 파일 화면 나오게 수정
  • 히스토리는 없고 오늘 미션 있을 때 운동 뷰 아래쪽에 흰색 파일 나오게 수정
  • 오늘 미션 인증 사진 나오게 수정
  • 신발 이미지 투명도 제거한 이미지로 수정
  • 아이콘, 상대방 운동 버블 상대방 기준으로 맞춤
  • 인증 사진 상자 꽉차게 수정, 모서리 둥글게 수정
  • ? 눌렀을 때 말풍선 나오게 만들기

📚 Reference 혹은 궁금한 사항들

? 눌렀을 때 팝업 나오는거 아직 추가 안했어요
추가했습니다~

@hyunjium hyunjium added feat 구현 및 개선 Pull Request 🔥 pr 날릴때 현지 🐱 엄니 QA qa 관련 작업 labels Apr 3, 2024
@hyunjium hyunjium added this to the 2차 스프린트 milestone Apr 3, 2024
@hyunjium hyunjium self-assigned this Apr 3, 2024
@hyunjium hyunjium marked this pull request as ready for review April 3, 2024 15:40
@@ -51,7 +51,7 @@
<service
android:name=".presentation.home.service.StepCountService"
android:exported="false"
android:foregroundServiceType="health" />
android:foregroundServiceType="health|dataSync" />
Copy link
Contributor

Choose a reason for hiding this comment

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

@Jokwanhee 이거 health로 둬도 되죠?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@l2zh
네 health로 테스트해봤는데 진행해도 괜찮을 것 같아요!

Copy link
Collaborator

@Jokwanhee Jokwanhee left a comment

Choose a reason for hiding this comment

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

고생했습니다 ~ 👍
그때 home 쪽 디버깅한 부분이 남아있었네요.. 그래서 버그가 좀 있었을 듯 합니다 ㅠ.ㅠ

android:layout_height="0dp"
android:layout_marginStart="8dp"
android:layout_marginEnd="16dp"
android:background="@drawable/shape_rectangle_radius8"
android:clipToOutline="true"
Copy link
Collaborator

Choose a reason for hiding this comment

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

clipToOutline 이 이미지 둥글게 만드는 속성으로 알고 있는데, api 하위 호환성이 없는 걸로 알고 있어서 coil 같은 이미지 라이브러리를 사용하는 것을 권장합니다!

Copy link
Collaborator Author

@hyunjium hyunjium Apr 4, 2024

Choose a reason for hiding this comment

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

coil 사용해보겠습니다~ 감사합니다~

Copy link
Contributor

@l2zh l2zh left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@l2zh l2zh merged commit 15f0ddb into develop Apr 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 구현 및 개선 Pull Request 🔥 pr 날릴때 QA qa 관련 작업 현지 🐱 엄니
Projects
Development

Successfully merging this pull request may close these issues.

[feat] 운동 뷰 QA 1차 수정
3 participants