feat: 위시 아이템 INCOMPLETE 상태 대응 - #520
Merged
Merged
Conversation
- 서버가 추출 결과를 일부만 채웠을 때 INCOMPLETE 를 내려준다 (TeamPiKi/core#945). 기존 코드는 FAILED·PENDING·PROCESSING 이 아니면 전부 정상 카드로 그려, 이름·가격이 빈 칸인 카드가 보이고 채우라는 유도가 없었다 - 위시 그리드에서 INCOMPLETE 를 FAILED 와 같은 편집 유도 카드로 보내되 문구만 "일부만 가져왔어요" 로 가른다 - name·price 를 nullable 로 바꾸면서 카드 컴포넌트도 nullable 을 받게 했다. INCOMPLETE 를 앞에서 걸러 실제로 빈 값이 정상 카드에 들어가지는 않는다 - 토너먼트: 담기 후보에서 INCOMPLETE 를 제외하고(서버가 출전을 막는다), 바스켓에서는 클릭 가능하게 둔다(값을 채워야 하므로)
Closed
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Important Review available on request
Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
왜
서버가 추출 결과를 일부만 채웠을 때
INCOMPLETE상태로 내려준다 (TeamPiKi/core#945). 클라는 이 값을 몰라서FAILED/PENDING/PROCESSING이 아니면 전부 정상 카드로 그렸다.그래서 이름과 가격이 빈 칸인 카드가 보이고(
formatPrice가 숫자를 못 찾아 빈 문자열 반환), 사용자에게 채우라는 유도가 없다.무엇을
ITEM_STATUS에INCOMPLETE추가,ItemT의name·price를 nullable 로INCOMPLETE를 앞에서 걸러내므로 실제로 빈 값이 정상 카드에 들어가지는 않는다문구만 다를 뿐 기존 실패 카드 흐름을 그대로 재사용해 변경을 최소화했다.
참고
알림 타입(
ITEM_PARSING_INCOMPLETE)과 SSE status 대응은 #519 로 분리했다. 두 PR 은 파일이 겹치지 않아 독립적으로 머지할 수 있다.연관 이슈