Skip to content

[Refactor] db이전#493

Merged
dolmeengii merged 2 commits into
Kernel360:developfrom
miniseung:refactor/DBmove
Aug 9, 2025
Merged

[Refactor] db이전#493
dolmeengii merged 2 commits into
Kernel360:developfrom
miniseung:refactor/DBmove

Conversation

@miniseung
Copy link
Copy Markdown
Collaborator

@miniseung miniseung commented Aug 7, 2025

🚀 풀 리퀘스트 제안

📋 작업 내용

수정한 내용이나 추가한 기능에 대해 자세히 설명해 주세요.

🔧 변경 사항

  • 📃 README.md
  • 📦 package.json
  • 🔥 파일 삭제
  • 🧹 그 외 ex) .gitignore 등

주요 변경 사항을 요약해 주세요.

📸 스크린샷 (선택 사항)

수정된 화면 또는 기능을 시연할 수 있는 스크린샷을 첨부해 주세요.

📄 기타

추가적으로 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.

Sourcery 요약 (Summary by Sourcery)

Next.js 설정 및 사용자 프로필 컴포넌트 전반에 걸쳐 Supabase 스토리지 도메인을 새 프로젝트로 업데이트

개선 사항:

  • Next.js 이미지 설정에서 기존 Supabase 프로젝트 호스트 이름을 새 도메인으로 교체
  • my-profile 및 other-profile 컴포넌트의 profileImg URL을 새 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:

  • Replace old Supabase project hostname with the new domain in Next.js image configuration
  • Update profileImg URLs in my-profile and other-profile components to use the new Supabase storage domain

@sourcery-ai
Copy link
Copy Markdown
Contributor

sourcery-ai Bot commented Aug 7, 2025

검토자 가이드 (작은 PR에서는 접혀 있음)

검토자 가이드

이 PR은 Next.js 원격 이미지 구성을 업데이트하고 두 사용자 프로필 구성 요소에서 하드코딩된 프로필 이미지 URL을 업데이트하여 이미지 호스팅을 이전 Supabase 프로젝트에서 새 프로젝트로 마이그레이션합니다.

파일 수준 변경 사항

변경 사항 세부 정보 파일
Next.js 이미지 구성을 새 Supabase 도메인으로 전환
  • 'domains' 배열에 새 도메인 추가
  • remotePatterns.hostname을 새 프로젝트로 업데이트
apps/web/next.config.ts
하드코딩된 프로필 이미지 URL 업데이트
  • MyProfile 구성 요소에서 이전 Supabase URL 교체
  • OtherProfile 구성 요소에서 이전 Supabase URL 교체
apps/web/src/components/user/my-profile.tsx
apps/web/src/components/user/other-profile.tsx

팁 및 명령

Sourcery와 상호 작용하기

  • 새 리뷰 트리거: 풀 리퀘스트에 @sourcery-ai review 댓글을 남기세요.
  • 토론 계속: Sourcery의 리뷰 댓글에 직접 답장하세요.
  • 리뷰 댓글에서 GitHub 이슈 생성: Sourcery에게 리뷰 댓글에 답장하여 이슈를 생성하도록 요청하세요. 리뷰 댓글에 @sourcery-ai issue를 답장하여 이슈를 생성할 수도 있습니다.
  • 풀 리퀘스트 제목 생성: 풀 리퀘스트 제목 어디에든 @sourcery-ai를 작성하여 언제든지 제목을 생성하세요. 풀 리퀘스트에 @sourcery-ai title 댓글을 남겨 언제든지 제목을 (재)생성할 수도 있습니다.
  • 풀 리퀘스트 요약 생성: 풀 리퀘스트 본문 어디에든 @sourcery-ai summary를 작성하여 원하는 위치에 언제든지 PR 요약을 생성하세요. 풀 리퀘스트에 @sourcery-ai summary 댓글을 남겨 언제든지 요약을 (재)생성할 수도 있습니다.
  • 검토자 가이드 생성: 풀 리퀘스트에 @sourcery-ai guide 댓글을 남겨 언제든지 검토자 가이드를 (재)생성하세요.
  • 모든 Sourcery 댓글 해결: 풀 리퀘스트에 @sourcery-ai resolve 댓글을 남겨 모든 Sourcery 댓글을 해결하세요. 모든 댓글을 이미 처리하여 더 이상 보고 싶지 않을 때 유용합니다.
  • 모든 Sourcery 리뷰 해제: 풀 리퀘스트에 @sourcery-ai dismiss 댓글을 남겨 기존의 모든 Sourcery 리뷰를 해제하세요. 새 리뷰를 처음부터 시작하고 싶을 때 특히 유용합니다. 새 리뷰를 트리거하려면 @sourcery-ai review 댓글을 잊지 마세요!

경험 사용자 정의

대시보드에 접속하여 다음을 수행하세요:

  • Sourcery가 생성한 풀 리퀘스트 요약, 검토자 가이드 등과 같은 검토 기능을 활성화하거나 비활성화합니다.
  • 검토 언어를 변경합니다.
  • 사용자 지정 검토 지침을 추가, 제거 또는 편집합니다.
  • 기타 검토 설정을 조정합니다.

도움 받기

Original review guide in English
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

This 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

Change Details Files
Switch Next.js image config to new Supabase domain
  • Added new domain to 'domains' array
  • Updated remotePatterns.hostname to the new project
apps/web/next.config.ts
Update hardcoded profile image URLs
  • Replaced old Supabase URL in MyProfile component
  • Replaced old Supabase URL in OtherProfile component
apps/web/src/components/user/my-profile.tsx
apps/web/src/components/user/other-profile.tsx

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

@miniseung 님, 변경 사항을 검토했습니다. 몇 가지 피드백이 있습니다:

  • next.config.ts와 컴포넌트 모두에서 URL이 중복되는 것을 방지하기 위해 Supabase 스토리지 도메인을 공유 환경 변수 또는 상수로 추출하세요.
  • Supabase 구성(엔드포인트 및 스토리지 경로)을 단일 모듈로 중앙 집중화하여 한 곳에서만 변경하면 되도록 고려하세요.
AI 에이전트용 프롬프트
이 코드 검토의 댓글을 처리해 주세요:
## 전체 의견
- next.config.ts와 컴포넌트 모두에서 URL이 중복되는 것을 방지하기 위해 Supabase 스토리지 도메인을 공유 환경 변수 또는 상수로 추출하세요.
- Supabase 구성(엔드포인트 및 스토리지 경로)을 단일 모듈로 중앙 집중화하여 한 곳에서만 변경하면 되도록 고려하세요.

Sourcery는 오픈 소스에 무료입니다 - 저희 리뷰가 마음에 드시면 공유를 고려해 주세요 ✨
제가 더 유용해지도록 도와주세요! 각 댓글에 👍 또는 👎를 클릭해 주시면 피드백을 통해 리뷰를 개선하겠습니다.
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.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@dolmeengii dolmeengii merged commit faf9dad into Kernel360:develop Aug 9, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants