You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
TeamPiKi/client#727: 위시 페이지 경로 개편 epic에 앱 공유 시트 경로 갱신 단계가 포함되어 있습니다.
Possibly related PRs
TeamPiKi/client#785: 웹 위시 경로를 /wish로 이전하며, 이 변경은 앱의 공유 시트와 native intent 경로를 같은 값으로 갱신합니다.
Suggested labels:fix, refactor
Suggested reviewers:kanghaeun
Merge Risk: Minimal · up to 79699
No actionable merge-blocking issue is established. Release the app only after the prerequisite web migration is deployed, as specified, and complete the planned device verification.
Pre-merge checks: 4 · 1
Failed checks: 1 warning
Check name
Status
Explanation
Resolution
Docstring Coverage
Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documenta…
Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed checks: 4 passed
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
제목 "feat: 앱 공유 시트 위시 경로 /wish 로 갱신"은 변경 사항의 핵심을 설명합니다. 공유 시트와 공유 인텐트 흐름에서 위시 경로를 /archive/wish에서 /wish로 바꾸는 것이 주된 변경이며, diff의 useShareIntent.ts, ShareBottomSheet.tsx, +native-intent.ts 변경과 일치합니다. …
Linked Issues check
직접 연결된 #735의 경로 요구사항을 충족합니다. ShareBottomSheet.tsx는 위시 목록과 상세 경로를 각각 /wish, /wish/${wishId}로 엽니다. useShareIntent.ts의 WISHLIST_PATH는 /wish이며, 다른 페이지에서 이 경로로 이동합니다. +native-intent.ts의 예시 주석…
Out of Scope Changes check
변경 범위는 #735 및 현재 PR 설명과 일치합니다. 위시 경로 변경, 공유 인텐트 경로 갱신, 개발용 프리뷰 라우트 삭제, 프리뷰 전용 export 정리가 모두 해당 작업을 지원합니다. 브릿지 메시지나 웹의 기존 /archive/wish 리다이렉트 변경은 확인되지 않습니다.
Full details: Title check
Explanation
제목 "feat: 앱 공유 시트 위시 경로 /wish 로 갱신"은 변경 사항의 핵심을 설명합니다. 공유 시트와 공유 인텐트 흐름에서 위시 경로를 /archive/wish에서 /wish로 바꾸는 것이 주된 변경이며, diff의 useShareIntent.ts, ShareBottomSheet.tsx, +native-intent.ts 변경과 일치합니다. 개발용 미리보기 화면 삭제와 타입 export 제거는 제목에 없지만 주된 변경을 가리는 수준은 아닙니다.
Full details: Linked Issues check
Explanation
직접 연결된 #735의 경로 요구사항을 충족합니다. ShareBottomSheet.tsx는 위시 목록과 상세 경로를 각각 /wish, /wish/${wishId}로 엽니다. useShareIntent.ts의 WISHLIST_PATH는 /wish이며, 다른 페이지에서 이 경로로 이동합니다. +native-intent.ts의 예시 주석도 /wish로 갱신했습니다. 프리뷰 라우트는 삭제했고, ShareBottomSheetView와 SheetStateT는 내부 선언으로 되돌렸습니다. 브릿지 메시지와 기존 웹 리다이렉트는 변경하지 않았습니다.
Full details: Docstring Coverage
Explanation
Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 3 functions across 3 files.
Resolution
Write docstrings for the functions missing them to satisfy the coverage threshold.
Finishing Touches
Generate docstrings
Commit to this branch
Create a new PR
Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
Run CodeRabbit Verify to test the changes in this pull request.
Recent review info
Run configuration
Source mode: Auto
Repository: Unavailable
Branch: Unavailable
File: No configuration file from this repository applied
Commit: Unavailable
Layers: Repository UI
Review profile: CHILL
Plan: Advanced
Run ID: 5a5455dd-cf50-40da-90ff-068707afd2c9
Commits
Reviewing files that changed from the base of the PR and between f4f88c6 and 7969977.
Files selected for processing: 4
apps/app/app/+native-intent.ts
apps/app/app/dev/share-sheet-preview.tsx
apps/app/components/ShareBottomSheet.tsx
apps/app/hooks/useShareIntent.ts
Files with no reviewable changes: 1
apps/app/app/dev/share-sheet-preview.tsx
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
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
APPGood for newcomersfeatureNew feature or request
1 participant
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.
작업 요약
/archive/wish에서/wish로 바꿉니다작업 세부 내용
머지 조건
/wish·/wish/:id가 없으면 새 앱이 404 로 감경로 전환
/archive/wish/wish/archive/wish/:id/wish/:id/archive/wish/wishShareBottomSheet의openHostApp경로와useShareIntent의WISHLIST_PATH값만 변경. 이동 로직은 그대로+native-intent는web쿼리를 그대로 넘기므로 주석의 예시 경로만 갱신/archive/wish리다이렉트는 구버전 앱용으로 유지. 이 PR 과 별개개발용 프리뷰 제거
app/dev/share-sheet-preview.tsx삭제./dev/share-sheet-preview라우트 소멸ShareBottomSheetView·SheetStateT를 파일 내부로 되돌림인수 조건 점검
openHostApp에/wish전달테스트
스크린샷
화면 변화 없음
연관 이슈
closes #735
Summary by CodeRabbit
/wish로 변경되었습니다. 이미 위시 목록에 있을 때는 공유한 항목을 바로 전송하며, 다른 화면에서는/wish로 이동합니다.