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: [2.0.0] 버전 업데이트에 따른 기능 추가 및 수정(카테고리 상세) #189

Merged
merged 8 commits into from
Feb 4, 2025

Conversation

stealmh
Copy link
Member

@stealmh stealmh commented Feb 3, 2025

#️⃣연관된 이슈

#183

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

2.0버전의 포킷 상세화면 작업을 했습니다.
기능적인 구현보다 디자인적 요소가 많았던 작업입니다.

  • 카테고리 내 링크가 없을 때 화면 분기 추가
  • 링크추가하기 float button
  • 즐겨찾기 상세화면
  • 이 외 기능 위치 수정
  • 필터링 다중 선택 가능하게끔 수정

스크린샷 (선택)

즐겨찾기 상세 카테고리 상세

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

로직상 문제는 없지만 2.0버전을 준비하면서 몇몇 API가 작동하지 않는 것을 확인했습니다. 또한 겹치는 API도 여럿 있는 상태고 하나하나 잡다가 더 지연될 것 같아서요.

ex) 즐겨찾기 컨텐츠 목록을 조회하기 위해 카테고리 내 컨텐츠 목록 조회를 사용해야 할까..?, 아니면 즐겨찾기 링크 모음 조회를 사용해야할까..?
(두 API의 response가 동일하기 때문에 ... 즐겨찾기 카테고리 목록을 굳이 뽑아써야 할까? 라는 의문이 들어서 내 컨텐츠 목록조회로 즐겨찾기 목록 뽑은 상태... 근데 빈값 반환)

2.0 작업들 전부 끝내고 릴리즈 전 QA때 한번에 잡아내는게 시간절약이 된다고 생각하는데 어떠신가요??

close #183

@stealmh stealmh added Design 🎨 디자인 작업 Fix 기능 수정 labels Feb 3, 2025
@stealmh stealmh requested a review from ShapeKim98 February 3, 2025 12:03
@stealmh stealmh self-assigned this Feb 3, 2025
Copy link
Contributor

@ShapeKim98 ShapeKim98 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!
저도 QA때 다 잡아내면 좋을 것 같습니다!
여담이지만, 이전에 저희가 뷰의 하위 뷰들을 구조화 할 때 Computed Property를 자주 썼었는데, 요즘 잘 안쓰시는거 같아서 지향해야할 이유가 있던건지 궁금합니다!

@stealmh
Copy link
Member Author

stealmh commented Feb 4, 2025

수고하셨습니다! 저도 QA때 다 잡아내면 좋을 것 같습니다! 여담이지만, 이전에 저희가 뷰의 하위 뷰들을 구조화 할 때 Computed Property를 자주 썼었는데, 요즘 잘 안쓰시는거 같아서 지향해야할 이유가 있던건지 궁금합니다!

맞슴다 저도 분리하는거 좋아합니다ㅎㅎㅎ
근데 전처럼 여유가 없어지다 보니까 계산속성으로 뺄 생각을 안해봤네요..네.. 반성함 😂😂😂

@stealmh stealmh force-pushed the feat/#183-pokit-detail branch from caf9e13 to 3f2cc13 Compare February 4, 2025 08:26
- 필터외부로 이동 (즐겨찾기 / 안읽음 / 정렬)
- 링크추가 float button
- 카테고리 케밥 버튼의 공유하기 외부로 이동
내부 필터 시트에서 작동하던 기능 외부로 이전
- 정렬 숨기기
- 링크 추가 float버튼 숨기기
- empty화면의 링크 추가 case 수정
@stealmh stealmh force-pushed the feat/#183-pokit-detail branch from 3f2cc13 to dc6a8d5 Compare February 4, 2025 08:30
@stealmh stealmh merged commit 842a4a9 into develop Feb 4, 2025
1 check passed
@stealmh stealmh linked an issue Feb 4, 2025 that may be closed by this pull request
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design 🎨 디자인 작업 Fix 기능 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[2.0.0] 포킷 상세 화면 리뉴얼
2 participants