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

feat: 마이페이지 3차 QA 대응 #418

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

feat: 마이페이지 3차 QA 대응 #418

wants to merge 8 commits into from

Conversation

yeonjeen
Copy link
Contributor

@yeonjeen yeonjeen commented Nov 2, 2024

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • 작품취향 키워드 클릭 막아두기
  • 보관함 정렬 선택 버튼 텍스트까지 범위 넓히기
  • 읽기상태 변경시 보관함 카운트 반영
  • 소개글 50자 정렬
  • 작품취향 소재 빠진 키워드 추가

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

KakaoTalk_Video_2024-11-03-01-32-04.mp4

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

보관함/내서재 onResume에 넣어 놓은 건 계속 고민을 해보겠습니다..! 다른 분들 코드 참고해서 ActivityResultLauncher 이용해서 해봤는데 잘 안되서 임시방편(?)으로 onResume에 넣어서 피알 올립니다,,,,

@yeonjeen yeonjeen added 🍯 [FEAT] 새로운 기능을 개발합니다. 👛 황후 연진 어느날 웹소소의 황후가 되어버렸다 labels Nov 2, 2024
@yeonjeen yeonjeen requested a review from a team November 2, 2024 16:36
@yeonjeen yeonjeen self-assigned this Nov 2, 2024
Copy link
Contributor

@librarywon librarywon left a comment

Choose a reason for hiding this comment

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

onResume만 잘 수정해봅시다! 고생하셨어요!

Comment on lines +164 to +167
override fun onResume() {
super.onResume()
myLibraryViewModel.loadMyLibrary()
}
Copy link
Contributor

Choose a reason for hiding this comment

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

r: 이거 저나 산군 붙잡고 같이 해보면 될 것 같아요

Copy link
Contributor

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

잔디냠냠

loadMyLibrary()
}

fun loadMyLibrary(){
Copy link
Contributor

Choose a reason for hiding this comment

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

c: updateMyLibrary ? 는 어때욥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍯 [FEAT] 새로운 기능을 개발합니다. 👛 황후 연진 어느날 웹소소의 황후가 되어버렸다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 마이페이지 3차 QA 대응
3 participants