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.

Sign up for GitHub

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

Jump to bottom

[FIX/#131] 친해지기 런칭데이 QA #134

Merged
merged 13 commits into from
Feb 11, 2024
Merged

[FIX/#131] 친해지기 런칭데이 QA #134

merged 13 commits into from
Feb 11, 2024

Conversation

unam98
Copy link
Collaborator

@unam98 unam98 commented Feb 10, 2024 •
edited
Loading

📌 개요

✨ 작업 내용

  • 이슈 참고해주세요!
  • Dialog 유틸 만들었습니다! 팝업 띄울 때 CommonDialogFragment 활용해주세요!

✨ PR 포인트

  • 카드 뒤집어지는 터치와 관심사 삭제하려고 누르는 터치가 focus가 겹칩니다. 카드 뒤집어지는 쪽으로 focus가 잡혀서 관심사 삭제가 안 눌리고 묻힙니다. 여기 터치 focus 분리 작업을 해주셔야 할 것 같습니다.
  • 컴포즈 쓰시는 분들은 첨부한 사진처럼 isModify, isModifyDetail 설정해주시면 됩니다.
  • 아직 1개 미만 삭제 제한, 2개 이하 시 추가 버튼 등은 구현 안 돼있습니다!

📸 스크린샷/동영상

KakaoTalk_Video_2024-02-11-04-53-43.mp4
스크린샷 2024-02-11 오전 4 59 45

@unam98
[fix/#131] 로그아웃/회원탈퇴 시 SP에 저장해뒀던 유저정보 clear
ac838b0 
- 토큰 만료 시에도 clear 작업 필요
@unam98
[fix/#131] 친해지기 유저 list 중복 제거
2e7770e
@unam98
[fix/#131] 친해지기 SP 제거
d69cc8d 
- datastore로 마이그레이션
- 로그아웃, 회원탈퇴 시 clear 확인 완료
- 친해지기 위치 기반 접속 시 api 호출하도록 수정
@unam98
[fix/#131] 토큰 refresh 실패 시 userInfo clear 로직 추가
824f45c
@unam98
[fix/#131] 공용 dialog 추가
a4f3ab9
@unam98
[fix/#131] ResMapper 수정
ff25a94
@unam98
[fix/#131] 소개서 "GOAL" textColor 캐릭터 색상 적용
6dafea1
@unam98
[fix/#131] 지역 text가 이미지 위로 올라오도록 elevation 조정
56eef2a 
- 이유는 모르겠으나 그림자 효과 없이 높이 배치만 조절되길래 별도의 outLineProvider 속성은 설정 안 해줌
@unam98
[fix/#131] 소개서 초기 load 시 기본값 뜨는 현상 제거
c600dd1
@unam98
[fix/#131] 커스텀 카드뷰 뒷면 item 간격 설정 및 접두어# 추가
011cec9
@unam98
[fix/#131] 관심사 삭제 기능 구현
1178991
@unam98
[fix/#131] 카드 뒷면 isModify, isModifyDetail 추가
ff0c44b
@unam98 unam98 added 우남 FEAT 새로운 기능 구현 FIX 버그 및 오류 해결 UI UI 관련 작업 labels Feb 10, 2024
@unam98 unam98 self-assigned this Feb 10, 2024
blueme0
blueme0 approved these changes Feb 11, 2024
View reviewed changes
Copy link
Member

@blueme0 blueme0 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~~~ 👍🏻👍🏻

app/src/main/java/com/teumteum/teumteum/presentation/familiar/introduce/IntroduceActivity.kt Outdated
@@ -50,16 +47,15 @@ class IntroduceActivity
}

private fun setUpListener() {
binding.btnStart.setOnSingleClickListener {
startFamiliarDialogActivity()
binding.btnStart.setOnClickListener {
Copy link
Member

Choose a reason for hiding this comment

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

따닥 방지를 위해 setOnSingleClickLister 부탁드림다 👍🏻

@unam98 unam98 merged commit 9142a88 into main Feb 11, 2024
1 check passed
blueme0
blueme0 reviewed Feb 12, 2024
View reviewed changes
Copy link
Member

@blueme0 blueme0 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

잘못 리뷰 달아서 다시 리뷰하러 갈게염

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blueme0 blueme0 blueme0 approved these changes

@kkk5474096 kkk5474096 Awaiting requested review from kkk5474096

@Mnseo Mnseo Awaiting requested review from Mnseo

Assignees

@unam98 unam98

Labels
FEAT 새로운 기능 구현 FIX 버그 및 오류 해결 UI UI 관련 작업 우남
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FIX] 친해지기 런칭데이 QA
2 participants
@unam98 @blueme0