Skip to content

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] 포킷 상세 화면 리뉴얼

3 participants