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

[ Fix ] 선배 약속 거절 페이지 에러 수정 #282

Merged
merged 8 commits into from
Oct 18, 2024

Conversation

se0jinYoon
Copy link
Collaborator

#️⃣ Related Issue

Closes #273

✅ Done Task

  • 약속 거절페이지 뒤로가기했을 때 이전 페이지 남아있도록 반영 (자세히 보기 페이지)
  • 거절 사유 바텀시트 핸들러 (위에 --- ) 추가
  • 거절 사유 클릭 시 배경에 바로바로 반영되던 거 수정 (적용하기 안 눌러도 뷰에 적용됐었�음)
  • 거절 사유 입력 textarea 글자 크기 수정 (variant = secondary 적용)
  • 약속 리스트 페이지 에러 발생시 에러페이지로 이동

☀️ New-insight

큰 이슈는 없었고 자잘자잘한 것들 수정해두었습니다 ~

💎 PR Point

📸 Screenshot

  • 이전에는 [거절페이지]에서 뒤로가기 누르면 [약속리스트] 페이지로 이동했었음
    -> [자세히보기] 페이지로 이동하도록 수정
2024-10-14.11.11.38.mov
  • 이전에는 클릭할 때 마다 뒤에 반영됐었음 -> 적용해야 반영되도록 수정
2024-10-14.11.11.57.mov
  • 바텀시트 --- 추가
image

@se0jinYoon se0jinYoon added 🛠 Fix 기존의 버그 수정 서진 labels Oct 14, 2024
@se0jinYoon se0jinYoon self-assigned this Oct 14, 2024
Copy link
Collaborator

@ijieun ijieun left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@j-nary j-nary left a comment

Choose a reason for hiding this comment

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

LGTM 고생하셧슴당 🚀🚀🚀

@se0jinYoon se0jinYoon merged commit 13da379 into develop Oct 18, 2024
1 check passed
@se0jinYoon se0jinYoon deleted the fix/#273/promiseListErrorStatus branch October 18, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Fix 기존의 버그 수정 size/m 서진
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[ Fix ] 선배 약속 거절 페이지 에러 수정
3 participants