Skip to content

Conversation

@jjangminii
Copy link
Collaborator

@jjangminii jjangminii commented Dec 29, 2025

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

  • close #

📄 Tasks

⭐ PR Point (To Reviewer)

📷 Screenshot

Summary by CodeRabbit

Chores

  • 소셜 로그인 리다이렉트 URL을 운영/개발 환경에 따라 자동으로 선택하도록 개선했습니다.
  • 환경별 설정 적용이 명확해져 배포 시 올바른 리다이렉트가 보장됩니다.
  • 검증 및 리다이렉트 URL 생성 흐름을 다듬어 가독성과 안정성을 향상시켰습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link

vercel bot commented Dec 29, 2025

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

Project Deployment Review Updated (UTC)
pinback-client-client Ready Ready Preview, Comment Dec 29, 2025 3:26pm
pinback-client-landing Ready Ready Preview, Comment Dec 29, 2025 3:26pm

@coderabbitai
Copy link

coderabbitai bot commented Dec 29, 2025

Walkthrough

프로덕션 여부(PROD)에 따라 Google 리다이렉트 URI를 환경 변수 VITE_GOOGLE_REDIRECT_URI_PROD 또는 VITE_GOOGLE_REDIRECT_URI_DEV로 선택하도록 로직을 변경했습니다.

Changes

Cohort / File(s) 변경 요약
소셜 로그인 환경 기반 URI 선택
apps/client/src/pages/onBoarding/components/funnel/step/SocialLoginStep.tsx
정적 VITE_GOOGLE_REDIRECT_URI 사용을 제거하고 import.meta.env.PROD 기반으로 VITE_GOOGLE_REDIRECT_URI_PROD 또는 VITE_GOOGLE_REDIRECT_URI_DEV를 선택하도록 변경; 불필요한 빈 줄 정리

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • jllee000
  • constantly-dev

Poem

🐰 프로덕션 길과 개발 길, 두 갈래를 알아보며
바람결에 환경 변수를 흩날리네 🌿
작은 토끼가 점프해 봄직한 곳에
리다이렉트 한 줄 길을 놓았네 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (4 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning 제목이 실제 변경사항(redirectUri 환경 기반 선택)과 맞지 않으며, 이미지 재저장과는 무관한 내용입니다. 제목을 실제 변경사항을 반영하도록 수정하세요. 예: Refactor(client): Google redirect URI 환경별 선택 적용
Description check ⚠️ Warning PR 설명이 템플릿 구조를 따르고 있지만 Related Issues, Tasks, PR Point 섹션이 모두 비어있습니다. 관련 이슈 번호, 수행한 작업 내용, 그리고 리뷰어를 위한 주요 사항을 작성하여 설명을 완성하세요.
Linked Issues check ⚠️ Warning PR의 실제 변경사항(SocialLoginStep.tsx의 redirectUri 환경 기반 선택)은 이슈 #25(Progress 컴포넌트 구현)의 요구사항과 전혀 무관합니다. 올바른 연결 이슈를 확인하거나, PR의 실제 목적에 맞는 이슈와 연결하세요.
Out of Scope Changes check ⚠️ Warning SocialLoginStep.tsx의 redirectUri 변경사항이 이슈 #25(Progress 컴포넌트 구현)와 완전히 무관하므로 범위를 벗어나있습니다. 이 변경사항과 관련된 올바른 이슈를 링크하거나, 이슈에 해당하는 변경사항으로 PR을 수정하세요.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/#227/redirecturi-edit

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7a22168 and e4458d5.

📒 Files selected for processing (1)
  • apps/client/src/pages/onBoarding/components/funnel/step/SocialLoginStep.tsx
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-07-15T20:00:13.756Z
Learnt from: constantly-dev
Repo: Pinback-Team/pinback-client PR: 80
File: apps/client/src/shared/components/ui/modalPop/ModalPop.tsx:36-41
Timestamp: 2025-07-15T20:00:13.756Z
Learning: In apps/client/src/shared/components/ui/modalPop/ModalPop.tsx, the InfoBox component uses hardcoded values for title, location, and icon URL as temporary test data. These should be replaced with dynamic data from props when implementing actual functionality and should be marked with TODO comments for future changes.

Applied to files:

  • apps/client/src/pages/onBoarding/components/funnel/step/SocialLoginStep.tsx
🔇 Additional comments (2)
apps/client/src/pages/onBoarding/components/funnel/step/SocialLoginStep.tsx (2)

8-10: PR 설명과 실제 변경사항 간의 불일치.

PR 설명(issue #25)에는 Progress 컴포넌트 구현이 명시되어 있으나, 실제 변경사항은 Google OAuth 리다이렉트 URI 선택 로직 수정입니다. PR 설명을 실제 변경사항에 맞게 업데이트하거나, 올바른 이슈와 연결해주세요.


8-10: 환경 변수 설정을 확인하세요.

import.meta.env.PROD를 사용한 환경별 리다이렉트 URI 선택 로직은 올바르게 구현되었습니다. 다만, VITE_GOOGLE_REDIRECT_URI_PRODVITE_GOOGLE_REDIRECT_URI_DEV 환경 변수가 프로덕션과 개발 환경에서 실제로 구성되어 있는지 확인이 필수입니다. 리포지토리에는 환경 변수 설정 파일이 포함되지 않으므로, Vercel 또는 해당 배포 플랫폼에서 이 변수들이 올바르게 설정되었는지 검증하세요.


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 and usage tips.

@github-actions
Copy link

github-actions bot commented Dec 29, 2025

✅ Storybook chromatic 배포 확인:
🐿️ storybook

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 512d124 and 7a22168.

⛔ Files ignored due to path filters (6)
  • packages/design-system/src/icons/source/chippi_profile.svg is excluded by !**/*.svg
  • packages/design-system/src/icons/source/tooltip_1.svg is excluded by !**/*.svg
  • packages/design-system/src/icons/source/tooltip_2.svg is excluded by !**/*.svg
  • packages/design-system/src/icons/source/tooltip_3.svg is excluded by !**/*.svg
  • packages/design-system/src/icons/source/tooltip_4.svg is excluded by !**/*.svg
  • packages/design-system/src/icons/source/tooltip_5.svg is excluded by !**/*.svg
📒 Files selected for processing (1)
  • apps/client/src/pages/onBoarding/components/funnel/step/SocialLoginStep.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: storybook
🔇 Additional comments (1)
apps/client/src/pages/onBoarding/components/funnel/step/SocialLoginStep.tsx (1)

9-11: 환경 변수 설정은 로컬 .env 파일에서 관리됩니다.

환경별 리다이렉트 URI 로직이 올바르게 구현되었습니다. import.meta.env.PROD는 Vite의 빌트인 불린이며, 프로덕션 빌드에서 VITE_GOOGLE_REDIRECT_URI_PROD, 개발 환경에서 VITE_GOOGLE_REDIRECT_URI_DEV를 선택하는 조건문이 정확합니다. 또한 코드에서 if (!clientId || !redirectUri) 체크로 누락된 환경 변수에 대한 에러 처리도 적절히 구현되어 있습니다.

.env 파일은 .gitignore에서 제외되므로, 각 개발자는 로컬에서 .env.local 또는 .env.development.local 파일에 VITE_GOOGLE_REDIRECT_URI_PRODVITE_GOOGLE_REDIRECT_URI_DEV를 정의해야 합니다.

Copy link
Member

@constantly-dev constantly-dev left a comment

Choose a reason for hiding this comment

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

👍

@jjangminii jjangminii changed the title Refactor: 이미지 재저장 Refactor(dsign-system): 이미지 재저장 Dec 29, 2025
@constantly-dev constantly-dev changed the title Refactor(dsign-system): 이미지 재저장 Refactor(design-system): 이미지 재저장 Dec 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants