Skip to content

[Bugfix/#388] 초대 수락 토큰 인코딩, 함수명 오타 및 import 경로 별칭으로 정리 - #393

Merged
jjjsun merged 3 commits into
developfrom
bugfix/#388
Aug 5, 2026
Merged

[Bugfix/#388] 초대 수락 토큰 인코딩, 함수명 오타 및 import 경로 별칭으로 정리#393
jjjsun merged 3 commits into
developfrom
bugfix/#388

Conversation

@jjjsun

@jjjsun jjjsun commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

🚨 관련 이슈

Closed #388

✨ 변경사항

  • 🐞 BugFix Something isn't working
  • 💻 CrossBrowsing Browser compatibility
  • 🌏 Deploy Deploy
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (storybook, jest, etc.)

✏️ 작업 내용

  • 초대 수락 API 함수명 오타 수정 (acceptInvitatonacceptInvitation)
  • 초대 토큰 경로에 encodeURIComponent 적용
  • 멤버 초대 로그인 필요 상태에 스크린 리더용 live region 추가
  • useDeleteMyAccount import 경로 @/ 별칭으로 정리

😅 미완성 작업

N/A

📢 논의 사항 및 참고 사항

main 배포 과정에서 발견된 초대 수락 관련 이슈 수정입니다.

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

Summary by CodeRabbit

  • 버그 수정
    • 초대 수락 기능의 오류를 수정하고 초대 토큰을 안전하게 처리합니다.
    • 초대 수락 페이지에서 로그인 필요 상태가 올바르게 안내됩니다.
  • 접근성
    • 로그인 필요 안내가 보조 기술에 자연스럽게 전달되도록 개선했습니다.

@jjjsun
jjjsun requested review from Seojegyeong and YermIm August 5, 2026 05:13
@jjjsun jjjsun self-assigned this Aug 5, 2026
@jjjsun jjjsun added the 🐞 BugFix Something isn't working label Aug 5, 2026
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

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

Next review available in: 51 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d5672eda-25fe-44bf-b863-bc8e6b74a440

📥 Commits

Reviewing files that changed from the base of the PR and between 4bd7c07 and b59ba23.

📒 Files selected for processing (1)
  • src/pages/workspace/InviteAcceptPage.tsx
📝 Walkthrough

Walkthrough

초대 수락 API의 함수명 오타를 수정하고 초대 토큰을 URL 인코딩합니다. 초대 수락 페이지의 호출부를 갱신하고 로그인 대기 상태에 접근성 속성을 추가합니다. 계정 삭제 훅의 import 경로를 별칭 경로로 변경합니다.

Changes

초대 수락 흐름

Layer / File(s) Summary
초대 수락 API 계약 수정
src/api/workspace/org.ts
공개 함수명을 acceptInvitation으로 수정하고 URL 경로의 tokenencodeURIComponent로 인코딩합니다.
초대 수락 페이지 연동 및 상태 표시
src/pages/workspace/InviteAcceptPage.tsx
수정된 API 함수를 import하고 mutation에서 호출합니다. 로그인 대기 컨테이너에 role="status"aria-live="polite"를 추가합니다.

계정 삭제 훅 import 경로

Layer / File(s) Summary
계정 삭제 훅 import 경로 정리
src/hooks/auth/useDeleteMyAccount.ts
useCoreMutation import 경로를 상대 경로에서 @/hooks/customQuery로 변경합니다.

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

Possibly related PRs

Suggested labels: 🔨 Refactor

Suggested reviewers: yermim, seojegyeong

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 초대 수락 토큰 인코딩, 함수명 오타 수정, import 경로 정리라는 주요 변경 사항을 명확하게 요약합니다.
Description check ✅ Passed 관련 이슈, 변경 유형, 작업 내용, 미완성 작업, 참고 사항을 모두 작성하여 템플릿 요구 사항을 충족합니다.
Linked Issues check ✅ Passed PR은 이슈 #388의 코드 품질 및 잠재 버그 피드백 반영 목표에 맞춰 함수명, 토큰 인코딩, 접근성, import 경로를 수정합니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항이 이슈 #388 및 PR 목표에 명시된 초대 수락 버그 수정과 코드 정리 범위에 포함됩니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bugfix/#388

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.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

📚 Storybook 배포 완료

항목 링크
📖 Storybook https://69a147b60a56365d9e2185ef-rjxgaojglq.chromatic.com/
🔍 Chromatic https://www.chromatic.com/build?appId=69a147b60a56365d9e2185ef&number=551

@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 `@src/pages/workspace/InviteAcceptPage.tsx`:
- Around line 201-205: InviteAcceptPage의 카운트다운 컨테이너에서 role="status"와
aria-live="polite"를 제거하고, 변경되지 않는 정적 안내 문장만 별도의 라이브 영역으로 감싸세요. countdown 표시 자체는
일반 텍스트 요소로 유지해 매초 전체 상태가 다시 읽히지 않도록 하세요.
🪄 Autofix

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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 3de6378c-882b-44ce-b029-cdd4a6dbf073

📥 Commits

Reviewing files that changed from the base of the PR and between a75cf48 and 4bd7c07.

📒 Files selected for processing (3)
  • src/api/workspace/org.ts
  • src/hooks/auth/useDeleteMyAccount.ts
  • src/pages/workspace/InviteAcceptPage.tsx

Comment thread src/pages/workspace/InviteAcceptPage.tsx Outdated

@YermIm YermIm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

P4: 확인했습니다!

@jjjsun
jjjsun merged commit ce280ad into develop Aug 5, 2026
3 checks passed
@jjjsun
jjjsun deleted the bugfix/#388 branch August 5, 2026 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐞 [BugFix] develop→main 머지 시 Coderabbit 추가 피드백 수정

2 participants