Skip to content

Conversation

@chldntjr1321
Copy link
Contributor

✅ PR 체크리스트

1. 코드 & 기능

  • 변수/함수 이름 이해 가능한지
  • 기능 정상 동작 & 콘솔 에러 없음

2. UI

  • UI 동작 및 레이아웃 확인

3. 컨벤션

  • 팀 컨벤션에 맞게 함수 이름을 정의했는지

🔗 이슈 번호

✨ 작업한 내용

  • baseFetcher.ts 삭제
  • coreFetch.ts 추가 -> 공통 fetcher
  • publicFetch.ts 추가 -> 비인증 api 전용 fetcher

💁 리뷰 요청 / 코멘트

💡 참고사항

@vercel
Copy link

vercel bot commented Jan 1, 2026

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

Project Deployment Review Updated (UTC)
global-nomad Ready Ready Preview, Comment Jan 1, 2026 4:04pm

@gemini-code-assist
Copy link

Summary of Changes

Hello @chldntjr1321, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 API 호출 로직을 리팩토링하여 기존의 baseFetcher.ts를 제거하고, 새로운 coreFetch.tspublicFetch.ts로 대체합니다. coreFetch.ts는 모든 fetch 요청의 핵심 로직을 처리하며, publicFetch.ts는 이를 기반으로 비인증 API 호출을 담당합니다. 이러한 변경은 API 호출 로직의 모듈성을 높이고, 인증 여부에 따라 fetcher를 분리하여 관리하기 위함입니다. 현재는 activities.ts 파일의 비인증 API 호출만 publicFetch로 전환되었으며, 다른 인증이 필요한 API 호출들은 향후 마이그레이션을 위해 임시로 주석 처리된 상태입니다.

Highlights

  • 기존 Fetcher 삭제: 기존의 baseFetcher.ts 파일이 삭제되었습니다.
  • 공통 Fetcher 추가: 새로운 공통 fetcher인 coreFetch.ts 파일이 추가되었습니다. 이 파일은 기본적인 fetch 로직, 타임아웃 처리 및 에러 핸들링을 담당합니다.
  • 비인증 API Fetcher 추가: 비인증 API 전용 fetcher인 publicFetch.ts 파일이 추가되었습니다. 이 fetcher는 coreFetch를 사용하여 공통 API URL을 구성합니다.
  • API 호출 마이그레이션: activities.ts 파일 내의 모든 API 호출이 baseFetcher에서 publicFetch로 마이그레이션되었습니다.
  • 인증 관련 API 임시 주석 처리: myActivities.ts, myNotifications.ts, myReservations.ts, oauth.ts, users.ts 파일에서 기존 baseFetcher를 사용하던 인증 관련 API 호출들이 향후 인증된 fetcher로의 마이그레이션을 위해 임시로 주석 처리되었습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

이 PR은 baseFetchercoreFetchpublicFetch로 분리하여 API 호출 로직을 리팩토링했습니다. 코드의 역할 분리가 명확해져 유지보수성이 향상된 점이 좋습니다. 다만, 새로 추가된 coreFetch의 에러 처리 방식, publicFetch의 문서화, 그리고 임시로 주석 처리된 코드들에 대해 몇 가지 개선점을 제안합니다. 아래 리뷰 코멘트를 확인해주세요.

Copy link
Member

@aahreum aahreum left a comment

Choose a reason for hiding this comment

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

리뷰 남겼습니다!! coreFetch 최고네요!! 👍👍👍

Copy link
Member

@aahreum aahreum left a comment

Choose a reason for hiding this comment

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

확인했습니다!!!
만들어주신 coreFetch 함수 기반으로 server fetch, bff fetch 작업해보겠습니다!!
감사합니다~!!!

Copy link
Contributor

@Jihyun0522 Jihyun0522 left a comment

Choose a reason for hiding this comment

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

코드 확인하고 리뷰 남깁니다!

구현이 꽤 복잡하고 어려워 보이는데, 정말 고생 많으셨습니다!👍
저도 우석님 코드 다시 보면서 많이 공부해보겠습니다!

@chldntjr1321 chldntjr1321 merged commit 6e41d07 into dev Jan 1, 2026
3 checks passed
@chldntjr1321 chldntjr1321 deleted the refactor/123/coreFetch branch January 1, 2026 16:05
@github-project-automation github-project-automation bot moved this from Backlog to Done in 글로벌 노마드 Jan 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[♻️ Refactor] 공통 및 비인증 API fetcher 구현

4 participants