Skip to content

fix: 내 토너먼트 페이지 게스트도 접근 가능하도록 변경 - #368

Merged
iOdiO89 merged 2 commits into
devfrom
feat/335-splash
Jul 23, 2026
Merged

fix: 내 토너먼트 페이지 게스트도 접근 가능하도록 변경#368
iOdiO89 merged 2 commits into
devfrom
feat/335-splash

Conversation

@iOdiO89

@iOdiO89 iOdiO89 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

작업 요약

  • 내 토너먼트(archive/tournament) 페이지에 게스트가 접근할 수 있도록 권한 검사 위치를 옮겼습니다.

작업 세부 내용

  • 기존에는 archive/layout.tsx에서 MEMBER 권한 검사를 하고 있어, 게스트가 archive 하위(토너먼트 포함) 전체에 접근하지 못했습니다.
  • 토너먼트 페이지는 게스트·멤버 모두 접근 가능해야 하므로, 권한 검사를 archive/wish/layout.tsx로 이동했습니다. (git mv — rename으로 추적)
  • 이제 MEMBER 권한 게이트는 위시(archive/wish/*) 서브트리에만 적용되고, 토너먼트는 게이트 없이 게스트·멤버 모두 접근 가능합니다.
  • 권한 검사가 위시 서브트리 한정으로 좁혀지면서 redirectPath?.startsWith(ROUTES.WISHLIST) 분기가 항상 참이 되므로, 비멤버는 곧바로 WishLoginRequired를 렌더하도록 단순화했습니다. (도달 불가능해진 redirect 분기와 미사용 ROUTES import 제거)

연관 이슈

closes #335

Summary by CodeRabbit

  • 버그 수정
    • 회원 유형이 아닌 사용자가 위시 아카이브에 접근할 경우 로그인 필요 안내가 표시되도록 개선했습니다.
    • 위시 아카이브 접근 시 잘못된 리디렉션이 발생하던 동작을 수정했습니다.

@iOdiO89 iOdiO89 self-assigned this Jul 22, 2026
@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
piki Ready Ready Preview, Comment Jul 23, 2026 4:58am

@gemini-code-assist

Copy link
Copy Markdown

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@github-actions
github-actions Bot requested review from soyeong0115 and ychany July 22, 2026 07:00
@github-actions

Copy link
Copy Markdown

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

@github-actions github-actions Bot added the fix Something isn't working label Jul 22, 2026
@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@iOdiO89, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 32 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 99512ab5-72f9-4ccc-8183-07c13bcd9847

📥 Commits

Reviewing files that changed from the base of the PR and between 0b0fd62 and 61148e1.

📒 Files selected for processing (1)
  • apps/web/src/app/archive/wish/layout.tsx
📝 Walkthrough

Walkthrough

위시 보관 레이아웃의 컴포넌트 이름을 WishArchiveLayout으로 변경하고, 회원이 아닌 사용자는 경로에 관계없이 WishLoginRequired를 렌더링하도록 접근 처리를 단순화했습니다.

Changes

위시 보관 레이아웃

Layer / File(s) Summary
위시 보관 레이아웃 및 접근 처리
apps/web/src/app/archive/wish/layout.tsx
Props 타입과 레이아웃 컴포넌트를 WishArchiveLayout으로 변경하고, 비회원 처리에서 조건부 리다이렉트를 제거해 WishLoginRequired를 즉시 렌더링하도록 수정했습니다. 기본 내보내기 대상도 새 컴포넌트로 변경했습니다.

Estimated code review effort: 2 (Simple) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목이 토너먼트 페이지의 게스트 접근 허용 변경을 정확히 요약합니다.
Linked Issues check ✅ Passed 권한 검사를 위시 하위 경로로 옮겨 /archive/tournament 게스트 접근이라는 #335의 분리 목적에 부합합니다.
Out of Scope Changes check ✅ Passed 변경은 권한 검사 위치 조정과 관련 정리만 포함하며 별도 범위 외 수정은 보이지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/335-splash

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@apps/web/src/app/archive/wish/layout.tsx`:
- Around line 28-29: In the wish layout’s member-check flow, keep only the
identity evaluation inside the try block, then move the WishLoginRequired early
return after the try/catch so the login prompt renders outside the
error-boundary-protected section.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f2b79f8d-ac91-403f-84e4-83166018998c

📥 Commits

Reviewing files that changed from the base of the PR and between 6247ffd and 0b0fd62.

📒 Files selected for processing (1)
  • apps/web/src/app/archive/wish/layout.tsx

Comment thread apps/web/src/app/archive/wish/layout.tsx Outdated
@iOdiO89
iOdiO89 merged commit cba393a into dev Jul 23, 2026
7 checks passed
@iOdiO89
iOdiO89 deleted the feat/335-splash branch July 23, 2026 06:37
iOdiO89 added a commit that referenced this pull request Jul 23, 2026
* fix: 내 토너먼트 페이지 게스트도 접근 가능하도록 변경

* fix: 위시 아카이브 로그인 유도 렌더를 try/catch 밖으로 분리
iOdiO89 added a commit that referenced this pull request Jul 23, 2026
* fix: proxy에서 세션쿠키로 적용되던 오류 수정

* feat: JWT 토큰 유틸 @piki/core 추가 (iat 비교·유효성·만료 계산)

* fix: 앱 토큰 동기화에 iat 비교 도입해 로그인 풀림 방지

* refactor: 웹 JWT 유틸 @piki/core로 이전

* fix: 토큰 갱신 응답 body를 camelCase로 파싱하도록 수정 (rotation 토큰 유실 해결)

* fix: 웹뷰 워밍업 후 쿠키 동기화 + 양방향 동기화 훅 통합 (iOS 쿠키 유실 방지)

* refactor: 토큰 갱신 응답 타입 AuthTokensT를 @piki/core로 추출

* chore: 미사용 jwt 타입 export 삭제

* fix: 내 토너먼트 페이지 게스트도 접근 가능하도록 변경 (#368)

* fix: 내 토너먼트 페이지 게스트도 접근 가능하도록 변경

* fix: 위시 아카이브 로그인 유도 렌더를 try/catch 밖으로 분리
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Epic] 홈/토너먼트/보관 화면 개선안 반영

1 participant