[Refactor] db이전#493
Merged
Merged
Conversation
Contributor
검토자 가이드 (작은 PR에서는 접혀 있음)검토자 가이드이 PR은 Next.js 원격 이미지 구성을 업데이트하고 두 사용자 프로필 구성 요소에서 하드코딩된 프로필 이미지 URL을 업데이트하여 이미지 호스팅을 이전 Supabase 프로젝트에서 새 프로젝트로 마이그레이션합니다. 파일 수준 변경 사항
팁 및 명령Sourcery와 상호 작용하기
경험 사용자 정의대시보드에 접속하여 다음을 수행하세요:
도움 받기Original review guide in EnglishReviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR migrates image hosting from the old Supabase project to a new one by updating Next.js remote image configuration and updating hardcoded profile image URLs in two user profile components. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
@miniseung 님, 변경 사항을 검토했습니다. 몇 가지 피드백이 있습니다:
- next.config.ts와 컴포넌트 모두에서 URL이 중복되는 것을 방지하기 위해 Supabase 스토리지 도메인을 공유 환경 변수 또는 상수로 추출하세요.
- Supabase 구성(엔드포인트 및 스토리지 경로)을 단일 모듈로 중앙 집중화하여 한 곳에서만 변경하면 되도록 고려하세요.
AI 에이전트용 프롬프트
이 코드 검토의 댓글을 처리해 주세요:
## 전체 의견
- next.config.ts와 컴포넌트 모두에서 URL이 중복되는 것을 방지하기 위해 Supabase 스토리지 도메인을 공유 환경 변수 또는 상수로 추출하세요.
- Supabase 구성(엔드포인트 및 스토리지 경로)을 단일 모듈로 중앙 집중화하여 한 곳에서만 변경하면 되도록 고려하세요.제가 더 유용해지도록 도와주세요! 각 댓글에 👍 또는 👎를 클릭해 주시면 피드백을 통해 리뷰를 개선하겠습니다.
Original comment in English
Hey @miniseung - I've reviewed your changes - here's some feedback:
- Extract the Supabase storage domain into a shared environment variable or constant to avoid duplicating the URL across both next.config.ts and your components.
- Consider centralizing your Supabase configuration (endpoints and storage paths) into a single module so changes only need to be made in one place.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Extract the Supabase storage domain into a shared environment variable or constant to avoid duplicating the URL across both next.config.ts and your components.
- Consider centralizing your Supabase configuration (endpoints and storage paths) into a single module so changes only need to be made in one place.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
dolmeengii
approved these changes
Aug 9, 2025
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🚀 풀 리퀘스트 제안
📋 작업 내용
수정한 내용이나 추가한 기능에 대해 자세히 설명해 주세요.
🔧 변경 사항
주요 변경 사항을 요약해 주세요.
📸 스크린샷 (선택 사항)
수정된 화면 또는 기능을 시연할 수 있는 스크린샷을 첨부해 주세요.
📄 기타
추가적으로 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.
Sourcery 요약 (Summary by Sourcery)
Next.js 설정 및 사용자 프로필 컴포넌트 전반에 걸쳐 Supabase 스토리지 도메인을 새 프로젝트로 업데이트
개선 사항:
Original summary in English
Summary by Sourcery
Update the Supabase storage domain to the new project across Next.js configuration and user profile components
Enhancements: