Skip to content

fix: 2.0.0 QA [2]#195

Merged
ShapeKim98 merged 10 commits intodevelopfrom
fix/#193-link-recommend
May 22, 2025
Merged

fix: 2.0.0 QA [2]#195
ShapeKim98 merged 10 commits intodevelopfrom
fix/#193-link-recommend

Conversation

@ShapeKim98
Copy link
Contributor

@ShapeKim98 ShapeKim98 commented May 19, 2025

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호
#193

📝작업 내용

링크 추천

  • 썸네일 아이콘 컬러 수정
  • +아이콘 수정
  • 페이지 배경색 수정
  • +버튼 눌렀을 때 플로우 수정
  • 전체보기 선택 / 개별 태그 선택 로직 수정

스크린샷 (선택)

💬리뷰 요구사항(선택)

  • 카테고리 목록 조회 API 로직을 들고와서 살짝 수정했는데 확인 부탁드려여~

close #193

@ShapeKim98 ShapeKim98 requested a review from stealmh May 19, 2025 14:23
@ShapeKim98 ShapeKim98 self-assigned this May 19, 2025
@ShapeKim98 ShapeKim98 added Fix 기능 수정 Bug 🔫 현재 발견된 버그를 수정하기 위함 labels May 19, 2025
Comment on lines 229 to 234
case .포킷선택_항목_눌렀을때(pokit: let pokit):
state.selectedPokit = pokit
return .none
case .포킷_추가하기_버튼_눌렀을때:
state.showSelectSheet = false
return shared(.async(.컨텐츠_추가_API), state: &state)
Copy link
Member

@stealmh stealmh May 19, 2025

Choose a reason for hiding this comment

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

여기 흐름이 좀 이해가 되지 않는데
선택항목 눌렀을 때 -> 카테고리 목록 중 하나를 누름
포킷 추가하기 버튼을 눌렀을 때 -> 카테고리 목록 최상단의 포킷 추가하기

저는 이렇게 이해를 했읍니다 그렇다면

포킷 선택항목을 눌렀을 때: selectedPokit에 포킷을 담고 시트 내려가면서 selectedPokit에 컨텐츠가 추가 되어야 하지 않나 생각이 드는데 맞나요??
포킷 추가하기 버튼 눌렀을 때: 카테고리 목록 최상단 (+카테고리 추가하기) 기능인 것 같은데 포킷 추가 화면이 나와야 할 것 같읍니다 로직 확인한번만 해주세여!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

여기 흐름이 좀 이해가 되지 않는데 선택항목 눌렀을 때 -> 카테고리 목록 중 하나를 누름 포킷 추가하기 버튼을 눌렀을 때 -> 카테고리 목록 최상단의 포킷 추가하기

저는 이렇게 이해를 했읍니다 그렇다면

포킷 선택항목을 눌렀을 때: selectedPokit에 포킷을 담고 시트 내려가면서 selectedPokit에 컨텐츠가 추가 되어야 하지 않나 생각이 드는데 맞나요?? 포킷 추가하기 버튼 눌렀을 때: 카테고리 목록 최상단 (+카테고리 추가하기) 기능인 것 같은데 포킷 추가 화면이 나와야 할 것 같읍니다 로직 확인한번만 해주세여!

이거 제가 로직을 잘못 이해 했었네요.. 수정하겠습니다!

Comment on lines +424 to +429
func categoryListFetch(request: BasePageableRequest) -> Effect<Action> {
return .run { send in
let categoryList = try await categoryClient.카테고리_목록_조회(request, false, true).toDomain()
await send(.inner(.카테고리_목록_조회_API_반영(categoryList: categoryList)), animation: .pokitDissolve)
}
}
Copy link
Member

Choose a reason for hiding this comment

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

👍👍👍

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.

고생많으셨읍니다! 카테고리 목록 로직은 잘 가져와서 적용해주신 것 같습니다.
나머진 다 괜찮고 리뷰 남겨드린것만 확인해주시면 될 것 같습니다~!

++
rebase하고 merge할 때 PokitCategorySettingView에 제가 빼먹은 코드가 하나 있어서.. 이거도 하나 추가해주세요🥲🥲

...
+ .ignoresSafeArea(.container, edges: .bottom)
  .task { await send(.뷰가_나타났을때).finish() }

@stealmh stealmh linked an issue May 20, 2025 that may be closed by this pull request
21 tasks
@ShapeKim98
Copy link
Contributor Author

@stealmh 반영했습니다 확인한번 해주세요!

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.

LGTM 👍👍

@ShapeKim98 ShapeKim98 merged commit c8a0d5d into develop May 22, 2025
1 check passed
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.

[2.0.0] QA

2 participants

Comments