Skip to content

Conversation

@ShapeKim98
Copy link
Contributor

#️⃣연관된 이슈

ex) #203

📝작업 내용

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

포킷 상세

  • 즐겨찾기 필터 설정 후 필터 해제 시 로직 문제 수정
  • empty 화면에 링크 추가 버튼 제거
  • empty 노출 시 즐겨찾기/안읽음 버튼 노출 필요

검색

  • 검색 empty 추가
  • 정렬 버튼 누락 수정

링크 추천

  • 키워드 선택 바텀시트와 선택한 키워드의 가나다 순 정렬 추가
  • 키워드 선택 바텀시트에서 선택된 키워드 누락 문제 수정

링크 카드

  • 링크 읽음 처리 수정 (2안: url로 이동하기 전에 링크 상세 조회 API를 한번 호출 후 이동)

링크 추가

  • 저장완로 토스트 노출시간 4초 변경 및 보기 텍스트 추가

미분류 편집

  • 선택 링크 포킷 이동 후 선택된 링크가 없을 시에 "링크 삭제", "포킷 이동" 이 활성화 되어있는 문제 수정

스크린샷 (선택)

💬리뷰 요구사항(선택)

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

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

  • 이번에 PokitLinkPopup, PokitCaution도 선언형으로 리팩토링 했습니다.

close #203

@ShapeKim98 ShapeKim98 requested a review from stealmh September 9, 2025 12:41
@ShapeKim98 ShapeKim98 self-assigned this Sep 9, 2025
@ShapeKim98 ShapeKim98 added Fix 기능 수정 Bug 🔫 현재 발견된 버그를 수정하기 위함 labels Sep 9, 2025
stealmh
stealmh previously approved these changes Sep 14, 2025
Copy link
Member

@stealmh stealmh left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!
수정한 내용이 직관적이라 잘 이해된 것 같아요.
확장성있게 일부 리팩토링까지 센스쟁이입니다.
빌드 테스트 에러가 있어서 해결하고 제가 머지하겠습니다.

+onAction은 어떤상황에서 쓰이도록 만드신건가요? 궁금해요. 기존에 action을 인자로 받는것과 무슨차이가 있는지??

@stealmh
Copy link
Member

stealmh commented Sep 14, 2025

Device 문제는 해결을 했는데 인증서부분에서 계속 막히는걸로 봐서 Development 인증서가 만료되었는지 확인부탁!
최근에 Release용은 업데이트 한것으로 알고 있는데 요부분도 한번 확인해주세여! test build에서 막히고 testflight release는 정상작동할것으로 예상됨..

@stealmh stealmh self-requested a review September 14, 2025 12:01
@stealmh stealmh dismissed their stale review September 14, 2025 12:02

build fail

@ShapeKim98 ShapeKim98 merged commit 2ec5403 into develop Sep 21, 2025
1 check failed
@ShapeKim98
Copy link
Contributor Author

고생많으셨습니다! 수정한 내용이 직관적이라 잘 이해된 것 같아요. 확장성있게 일부 리팩토링까지 센스쟁이입니다. 빌드 테스트 에러가 있어서 해결하고 제가 머지하겠습니다.

+onAction은 어떤상황에서 쓰이도록 만드신건가요? 궁금해요. 기존에 action을 인자로 받는것과 무슨차이가 있는지??

팝업 클릭 액션이 필요한거와 필요하지 않는거와의 구분을 하고 싶었습니다. ㅎㅎ

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

Labels

Bug 🔫 현재 발견된 버그를 수정하기 위함 Fix 기능 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants