Skip to content

Conversation

@junglesub
Copy link
Member

@junglesub junglesub commented Jun 6, 2025

…and LandingPage; remove unnecessary line in ProfileRegistrationPage

✅ PR 체크리스트

  • 로컬 환경에서 작동 확인 (애러 없는지)
  • 제목에 작업 파트 추가 ([FE] 또는 [BE] Prefix 추가)

📌 관련 이슈


✨ 작업 내용

기존에는 기본 동아리로 바로 이동하는 방식이었지만 동아리 페이지가 추가되서 동아리로 이동되도록 수정

Summary by CodeRabbit

  • 버그 수정
    • 학생 인증, 로그인, 프로필 등록 후 이동 경로가 "/club/callein"에서 "/club"으로 변경되었습니다.
    • 관련 안내 메시지도 "club/callein"에서 "club"으로 수정되었습니다.

…LandingPage; remove unnecessary line in ProfileRegistrationPage
@junglesub junglesub self-assigned this Jun 6, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jun 6, 2025

Walkthrough

로그인 및 회원가입 후 이동 경로가 기존의 "/club/callein"에서 "/club"으로 변경되었습니다. 이 변경은 GoogleCallback, LandingPage, ProfileRegistrationPage 세 페이지에서 일관되게 적용되었으며, 로직이나 에러 처리에는 다른 변경이 없습니다.

Changes

파일(들) 변경 요약
src/main/front/src/pages/GoogleCallback.tsx, src/main/front/src/pages/LandingPage.tsx, src/main/front/src/pages/ProfileRegistrationPage.tsx 로그인 및 회원가입 후 이동 경로를 "/club/callein"에서 "/club"으로 변경, 불필요한 빈 줄 제거 및 콜백 함수 포맷 변경(로직 변화 없음)

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant GoogleCallback
    participant LandingPage
    participant ProfileRegistrationPage
    participant Router

    User->>GoogleCallback: 구글 로그인 콜백 진입
    GoogleCallback->>Router: "/club"로 이동

    User->>LandingPage: 랜딩 페이지에서 로그인 시도
    LandingPage->>Router: "/club"로 이동 (JWT 토큰 존재 시)

    User->>ProfileRegistrationPage: 프로필 등록 완료
    ProfileRegistrationPage->>Router: "/club"로 이동
Loading

Assessment against linked issues

Objective Addressed Explanation
로그인 후 기본 동아리(/club/callein)로 이동하지 않고 동아리 페이지(/club)로 이동하도록 수정 (#223)

Assessment against linked issues: Out-of-scope changes

해당 변경사항 내에서 이슈 목적과 무관한 기능적 코드 변경은 발견되지 않았습니다.

Possibly related PRs

  • Fe/feat/183 implement mainpage #206: GoogleCallback.tsx에서 로그인 후 "/club/callein" 또는 "/register"로 조건부 이동을 구현하며, 본 PR과 동일 파일 및 맥락에서 반대의 라우팅 타겟을 다루고 있어 직접적으로 연관됨.

Suggested labels

fe

Suggested reviewers

  • ejPark43

Poem

🐰
로그인 토끼가 뛰어가네,
동아리방 향해 경로를 바꿔가네!
callein은 안녕, club으로 출발,
회원가입도, 콜백도 모두 한 방향!
코드의 숲에 봄바람 솔솔,
토끼는 오늘도 PR을 올려요!
🌱


📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e16df6a and b2e131e.

📒 Files selected for processing (3)
  • src/main/front/src/pages/GoogleCallback.tsx (1 hunks)
  • src/main/front/src/pages/LandingPage.tsx (1 hunks)
  • src/main/front/src/pages/ProfileRegistrationPage.tsx (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
  • GitHub Check: vitest
🔇 Additional comments (4)
src/main/front/src/pages/LandingPage.tsx (1)

17-17:

Details

✅ Verification successful

네비게이션 경로 변경이 올바르게 적용되었습니다.

"/club/callein"에서 "/club"으로의 경로 변경이 다른 파일들과 일관성 있게 적용되었습니다.

다음 스크립트를 실행하여 라우팅 설정을 확인해 주세요:


🏁 Script executed:

#!/bin/bash
# "/club" 경로가 라우터에 정의되어 있는지 확인
rg -A 3 -B 3 'path.*["\']/?club["\']' --type ts --type tsx

# 혹시 남아있는 "/club/callein" 참조가 있는지 확인  
rg '/club/callein' --type ts --type tsx

Length of output: 116


라우팅 설정 검증을 위해 스크립트 따옴표 오류를 수정했습니다. 아래 명령을 실행해 주세요:


🏁 Script executed:

#!/bin/bash
# "/club" 경로가 라우터에 정의되어 있는지 확인
rg -A 3 -B 3 "<Route[^>]*path=['\"]/club['\"]" --type ts --type tsx

# 혹시 남아있는 "/club/callein" 참조가 있는지 확인
rg "/club/callein" --type ts --type tsx

Length of output: 169


rg--type tsx 지정이 지원되지 않아, 대신 --glob 옵션으로 .ts/.tsx 파일을 검색하도록 수정했습니다. 아래 스크립트를 실행해 주세요.


🏁 Script executed:

#!/bin/bash
# 1) "/club" 경로가 라우터에 정의되어 있는지 확인
rg -B3 -A3 '<Route[^>]*path=[\'"]/club[\'"]' --glob '*.ts' --glob '*.tsx'

# 2) 남아있는 "/club/callein" 참조가 있는지 확인
rg '/club/callein' --glob '*.ts' --glob '*.tsx'

Length of output: 116


스크립트 따옴표 문제를 해결했습니다. 아래 명령을 실행해 주세요.


🏁 Script executed:

#!/bin/bash
# 1) "/club" 경로가 라우터에 정의되어 있는지 확인
rg -B3 -A3 "<Route[^>]*path=['\"]/club['\"]" --glob "*.ts" --glob "*.tsx"

# 2) 남아있는 "/club/callein" 참조가 있는지 확인
rg "/club/callein" --glob "*.ts" --glob "*.tsx"

Length of output: 127


라우팅 설정 파일이 .js/.jsx에도 있을 수 있어, 해당 확장자까지 포함해 다시 검색해 주세요.


🏁 Script executed:

#!/bin/bash
# 1) 모든 확장자에서 <Route path=> 확인
rg -n "<Route[^>]*path\s*=\s*['\"][^'\"]+['\"]" --glob "*.{ts,tsx,js,jsx}"

# 2) createBrowserRouter나 BrowserRouter 사용 여부 확인
rg -n "createBrowserRouter|BrowserRouter" --glob "*.{ts,tsx,js,jsx}"

Length of output: 651


라우터 설정 내 실제 경로 정의를 확인해야 합니다. createBrowserRouter 호출 부근을 출력해 주세요:


🏁 Script executed:

#!/bin/bash
# createBrowserRouter 안의 경로 배열을 확인
rg -n 'createBrowserRouter' -A50 -B5 src/main/front/src/main.tsx

Length of output: 2530


"/club" 경로가 라우터에 정의되어 있으며, 더 이상 "/club/callein" 참조가 없습니다.

  • src/main/front/src/main.tsx 53번째 줄: { path: "/club", element: <ClubListPage /> }
  • 코드베이스 전역에서 /club/callein 참조 검색 결과 없음

따라서 navigate("/club") 사용이 올바릅니다.

src/main/front/src/pages/ProfileRegistrationPage.tsx (2)

152-152: 네비게이션 경로가 일관성 있게 업데이트되었습니다.

프로필 등록 완료 후 "/club" 경로로 이동하도록 변경된 것이 다른 파일들과 일치합니다.


187-189: 코드 스타일이 개선되었습니다.

onUploaded callback이 멀티라인 화살표 함수로 변경되어 가독성이 향상되었습니다. 기능적으로는 동일합니다.

src/main/front/src/pages/GoogleCallback.tsx (1)

84-85: 출력 메시지와 네비게이션 경로가 일관성 있게 업데이트되었습니다.

학생 인증 성공 후 사용자에게 표시되는 메시지와 실제 네비게이션 경로가 모두 "/club"으로 통일되어 사용자 경험이 개선되었습니다.


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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@github-project-automation github-project-automation bot moved this to Backlog in HANDONG-CMS Jun 6, 2025
@coderabbitai coderabbitai bot added the fe Frontend label Jun 6, 2025
@junglesub junglesub merged commit 0949cfd into main Jun 6, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in HANDONG-CMS Jun 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fe Frontend

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[BUG] 로그인 시 기본 동아리 설정 삭제

1 participant