Skip to content

[refactor] 가이드 페이지 이미지 변경 및 구조 개선#260

Merged
ff1451 merged 1 commit intodevelopfrom
259-refactor-가이드-페이지-이미지-변경-반영-및-구조-리팩토링
Apr 7, 2026

Hidden character warning

The head ref may contain hidden characters: "259-refactor-\uac00\uc774\ub4dc-\ud398\uc774\uc9c0-\uc774\ubbf8\uc9c0-\ubcc0\uacbd-\ubc18\uc601-\ubc0f-\uad6c\uc870-\ub9ac\ud329\ud1a0\ub9c1"
Merged

[refactor] 가이드 페이지 이미지 변경 및 구조 개선#260
ff1451 merged 1 commit intodevelopfrom
259-refactor-가이드-페이지-이미지-변경-반영-및-구조-리팩토링

Conversation

@ff1451
Copy link
Copy Markdown
Collaborator

@ff1451 ff1451 commented Apr 7, 2026

✨ 요약

- 가이드 페이지 이미지 경로를 신규 `.webp` 파일 기준으로 변경했습니다.
- `GUIDE_ITEMS` 하드코딩 배열에서 파일명 배열 기반으로 생성하도록 리팩토링했습니다.
- 이미지 base URL, duration 상수를 분리해 데이터 구조를 정리했습니다.
- `encodeURIComponent` 적용해 한글 파일명도 안전하게 URL로 생성되도록 수정했습니다.



😎 해결한 이슈



Summary by CodeRabbit

  • 최적화

    • 가이드 이미지 형식을 PNG에서 WebP로 업그레이드하여 파일 크기 감소 및 로딩 성능 향상
  • 개선사항

    • 가이드 항목의 데이터 구조 및 관리 방식 개선

@ff1451 ff1451 self-assigned this Apr 7, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 7, 2026

Walkthrough

src/pages/Guide/guideData.ts 파일의 GUIDE_ITEMS 배열이 리팩토링되었습니다. 기존 하드코딩된 배열을 GUIDE_IMAGE_FILE_NAMES 배열로부터 동적으로 생성하도록 변경되었으며, GUIDE_IMAGE_BASE_URL, GUIDE_ITEM_DURATION 상수가 추가되고, getGuideImageUrl() 함수로 이미지 URL을 구성합니다. 이미지 포맷은 .png에서 .webp로 변경되었고, GUIDE_ITEMS에 명시적 타입 어노테이션이 적용되었습니다.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 가이드 페이지의 이미지 변경과 구조 개선이라는 주요 변경사항을 명확히 요약하고 있습니다.
Linked Issues check ✅ Passed PR의 변경사항이 이슈 #259의 요구사항을 충족합니다: 리디자인된 .webp 이미지 적용, 하드코딩된 배열의 구조 개선, encodeURIComponent를 통한 안전한 URL 변환.
Out of Scope Changes check ✅ Passed 모든 변경사항이 이슈 #259의 범위 내에 있습니다: 이미지 경로 변경, 데이터 구조 리팩토링, URL 인코딩 처리만 포함되어 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 259-refactor-가이드-페이지-이미지-변경-반영-및-구조-리팩토링

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.

Copy link
Copy Markdown
Contributor

@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.

🧹 Nitpick comments (1)
src/pages/Guide/guideData.ts (1)

7-12: 이미지 호스트를 환경 변수로 분리하는 걸 권장합니다.

stage-static 도메인이 상수에 고정되어 있어 환경 전환 시 수정 포인트가 생깁니다. import.meta.env(또는 공통 config)로 분리하면 운영/스테이징 관리가 더 안전해집니다.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/pages/Guide/guideData.ts` around lines 7 - 12, GUIDE_IMAGE_BASE_URL is
hardcoded to the stage domain; change it to read from an environment variable
(e.g. import.meta.env.VITE_GUIDE_IMAGE_BASE_URL or your project's config) and
provide a sensible fallback to the existing URL. Update the constant
GUIDE_IMAGE_BASE_URL to use the env value, keep getGuideImageUrl(fileName:
string) as-is (it will reference the new constant), and ensure
GUIDE_IMAGE_FILE_NAMES remains unchanged; validate any build/env docs so
staging/production use the correct env key.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/pages/Guide/guideData.ts`:
- Around line 7-12: GUIDE_IMAGE_BASE_URL is hardcoded to the stage domain;
change it to read from an environment variable (e.g.
import.meta.env.VITE_GUIDE_IMAGE_BASE_URL or your project's config) and provide
a sensible fallback to the existing URL. Update the constant
GUIDE_IMAGE_BASE_URL to use the env value, keep getGuideImageUrl(fileName:
string) as-is (it will reference the new constant), and ensure
GUIDE_IMAGE_FILE_NAMES remains unchanged; validate any build/env docs so
staging/production use the correct env key.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: aa553643-22a4-4c41-829f-31c41df8e587

📥 Commits

Reviewing files that changed from the base of the PR and between c4d304a and 1ac0cc5.

📒 Files selected for processing (1)
  • src/pages/Guide/guideData.ts

@ff1451 ff1451 merged commit e4ebf17 into develop Apr 7, 2026
2 checks passed
@ff1451 ff1451 deleted the 259-refactor-가이드-페이지-이미지-변경-반영-및-구조-리팩토링 branch April 7, 2026 09:32
ff1451 added a commit that referenced this pull request Apr 21, 2026
* [배포] 이미지 전처리, 광고 기능, 하단바 리디자인, 인앱 알림 페이지 및 토스트 프로덕션 배포 (#230)

* 205 feat 이미지 전처리 기능 구현 (#206)

* feat: 전처리 로직 및 WebWorker 구현

* feat: 전처리 적용 및 preview 동시성 제어 로직 추가

* refactor: 리뷰 반영

* [hotfix] 하단바 너비 수정 (#208)

* hotfix: 하단바 너비 수정

* chore: 불필요한 값 제거

* refactor: 고정 gap 제거

* Reapply "[feat] 광고 배너 추가 (#200)"

This reverts commit c51ec85.

* [feat] 하단바 리디자인 (#213)

* chore: asset 추가

* feat: 하단바 리디자인 반영 및 레이아웃 수정

* [refactor] 광고 카드 레이아웃 밀림 수정 (#215)

* refactor: 광고 개수 측정 시기 변경 및 기본값 제거

* feat: 스켈레톤 UI 추가

* feat: 인앱 알림 페이지 및 토스트 구현 (#217)

* feat: 알림 API 및 스트림 기반 추가

* feat: 인앱 알림 레이어 추가

* feat: 알림 페이지 및 헤더 진입 구현

* fix: 알림 스트림 401 재시도 조건 정리

* fix: 알림 목록 이동 차단 제거

* refactor: 알림 공용 훅 위치 정리

* fix: 알림 재연결 캐시 동기화 추가

* fix: 알림 목록 토스트 큐 누적 방지

* fix: 알림 읽음 카운트 감소 조건 보강

* [refactor] 도메인별 TanStack Query 훅 정리 (#219)

* chore: pwa용 이미지 제거

* refactor: auth 도메인 쿼리와 뮤테이션 정리

* refactor: council과 schedule 조회 훅 정리

* refactor: chat과 notification 캐시 처리 정리

* refactor: club 조회와 지원 플로우 정리

* refactor: manager 도메인 캐시 처리 정리

* refactor: studyTime 도메인 쿼리와 뮤테이션 정리

* refactor: 광고와 업로드 도메인 훅 정리

* [refactor] mutaton query 및 hook 추가 수정 (#221)

* refactor: auth와 user myInfo 훅 정리

* refactor: club과 schedule 조회 훅 정리

* refactor: chat과 notification 훅 구조 정리

* refactor: club 지원 뮤테이션 훅 정리

* refactor: manager 뮤테이션 훅 구조 정리

* refactor: mutation 훅 cache 정리

* refactor: 컨벤션 통일

* refactor: isRead 조건 정리

* fix: 채팅 스크롤 문제 수정

* refactor: 불필요한 코드 제거

* [fix] 모바일 환경 입력창과 키보드 간의 간격이 큰 문제 수정 (#223)

* chore: 가공용 safeArea 변수 선언

* refactor: 고정 패딩 값 수정 및 safeArea 적용 변경

* feat: 키보드 활성화 감지 및 safeArea 적용 여부 기능 추가

* refactor: 매직넘버 상수화 및 가로모드 처리

* [fix] 키보드 활성화 시 화면 흔들림 문제 수정 (#225)

* refactor: 채팅 viewport 훅 네이밍 정리

* refactor: viewport 높이 잠금 훅 적용 시점 조정

* [fix] 키보드 활성화 시 채팅 화면 전체가 흔들리는 문제 수정 (#227)

* refactor: 채팅 viewport 훅 네이밍 정리

* refactor: viewport 높이 잠금 훅 적용 시점 조정

* fix: 채팅 화면 스크롤 잠금으로 키보드 흔들림 완화

* fix: 입력 포커스 중 viewport offset 고정 (#229)

* fix: 문서 루트 스크롤 잠금으로 빈 공간 잔류 방지 (#232)

* [fix] 키보드 활성화 시 채팅 화면 상단 고정이 깨지고 빈 공간이 남는 문제 수정 (#234)

* fix: 채팅 화면 상단 고정 깨짐과 빈 공간 잔류 수정

* refactor: 라우트 조건 수정

* fix: 문서 스크롤 위치 감지 보강

* refactor: 입력 요소 판별 유틸과 스크롤 주석 정리

* [fix] 키보드 활성화 시 채팅 화면에서 문서 스크롤이 발생하는 문제 수정 (#236)

* fix: 채팅 문서 스크롤 제스처 차단

* fix: 입력 요소 터치 동작 예외 처리

* [fix] 키보드 활성화 시 채팅방이 마지막 메시지 위치를 유지하지 못하는 문제 수정 (#238)

* fix: 키보드 활성화 시 채팅 하단 정렬 유지

* refactor: 채팅 리사이즈 관찰 안정화

* fix: mypage 연계 약관 페이지 뒤로가기 수정 (#240)

* refactor: alias import 경로 정리

* fix: query 설정과 suspense 분기 정리

* refactor: 관리자 화면 스타일 유틸 정리

* fix: 이미지 전처리 예외 처리 보강

* fix: 헤더와 회비 화면 동작 정리

* fix: 공통 유틸 안정성 개선

* fix: 이미지 전처리 실패 처리를 보정

* fix: 모집 공고 저장 후 설정 반영 순서 조정

* fix: 부원 직책 변경 실패 처리를 보강

* fix: 약관 링크 접근성을 개선

* fix: 공통 쿼리와 유틸 안정성을 보완

* [feat] 동적 버전 정보 표시 구현 (#211)

* feat: 동적 버전 정보 표시 구현

* refactor: 버전 정보 미 존재시 v 표시 제거

* [feat] 메인화면 동아리 카드 디자인 수정 반영 (#242)

* feat: 메인화면 동아리 카드 디자인 수정

* chore: 하단바 아이콘 수정

* refactor: 코드래빗 리뷰 반영

* refactor: and 연산자로 변경

* apiClient 코드 중복 제거 및 네이티브 브릿지 인증 동기화 중앙화 (#244)

* refactor: apiClient 코드 중복 제거 및 네이티브 브릿지 인증 동기화 중앙화

* refactor: body 직렬화 가드를 plain object/array로 한정

* fix: body 읽기 중 AbortError가 ParseError로 오분류되는 문제 수정

* [refactor] 에러 처리 유틸 및 utils 구조 정리 (#246)

* refactor: 에러 처리 유틸 및 공통 토스트 흐름 정리

* refactor: utils 폴더 구조를 역할별로 정리

* refactor: 코드래빗 리뷰 반영

* refactor: 코드래빗 리뷰 반영

* Update src/pages/Home/components/HomeClubSection.tsx

* fix: 인증 세션 복구 흐름 정리

* fix: 홈 동아리 카드 레이아웃 정리

* [feat] 총동아리 페이지 리디자인 및 하단 오버레이 정리 (#249)

* refactor: 하단 오버레이 처리 공통화

* feat: 총동아리 페이지와 헤더 리디자인 반영

* fix: 채팅 하단 여백과 외부 링크 속성 수정

* refactor: 총동아리 헤더 설정 정리

* fix: 총동아리 상세 접근성과 스타일 보완

* [feat] 마이페이지 관리자 카드 분리 및 채팅 미확인 배지 반영 (#251)

* feat: 하단 채팅 배지 표시 및 조회 조건 보완

* refactor: 관리자 정보 카드 컴포넌트 분리

* feat: 채팅 페이지 리디자인 (#252)

* feat: 채팅 페이지 리디자인

* fix:tailwind 문법 수정

* fix: 코드 수정

* fix: 폰트 색상 및 위치 수정

* fix: 채팅방 사람수 정렬

* fix: 오타 수정

* chore: conflict 해결 중 누락된 부분 수정

* [refactor] 광고 렌더링 조건 수정 (#254)

* refactor: 광고 렌더링 조건 수정

* docs: 문서명 변경

---------

Co-authored-by: 박성주 <145267904+ParkSungju01@users.noreply.github.com>

* refactor: 가이드 페이지 이미지 변경 및 구조 개선 (#260)

* hotfix: 가이드 이미지 경로 변경

---------

Co-authored-by: 박성주 <145267904+ParkSungju01@users.noreply.github.com>
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.

[refactor] 가이드 페이지 이미지 변경 반영 및 구조 리팩토링

1 participant