Skip to content

[hotfix] 가이드 이미지 403 에러 수정#290

Merged
ff1451 merged 7 commits intodevelopfrom
hotfix/guide-image-403-fix
Apr 21, 2026
Merged

[hotfix] 가이드 이미지 403 에러 수정#290
ff1451 merged 7 commits intodevelopfrom
hotfix/guide-image-403-fix

Conversation

@ff1451
Copy link
Copy Markdown
Collaborator

@ff1451 ff1451 commented Apr 21, 2026

✨ 요약

가이드 이미지의 경로에 한글이 들어가있어 에러가 발생하는 문제를 수정합니다.



Summary by CodeRabbit

릴리스 노트

  • Documentation

    • Pull Request 작성 가이드 템플릿 추가
  • 일반 개선

    • 가이드 이미지 리소스 표준화

ff1451 and others added 7 commits April 1, 2026 23:01
* 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>
@ff1451 ff1451 self-assigned this Apr 21, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 21, 2026

Walkthrough

GitHub Pull Request 템플릿이 새롭게 추가되었습니다. 템플릿은 요약, 해결된 이슈 섹션을 포함하며 기본 구조를 제공합니다. 동시에 가이드 데이터에서 이미지 파일명이 한글 기반 네이밍(가이드1.webp, 순공시간_개인별.webp)에서 숫자 기반 네이밍(1.webp부터 5.webp)으로 변경되어, 생성되는 가이드 아이템의 이미지 URL이 업데이트됩니다.

Possibly related PRs

🚥 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의 주요 변경사항을 명확하게 설명합니다. 가이드 이미지 경로의 한글 제거로 인한 403 에러 수정이 핵심 변경사항이며, 제목이 이를 정확히 반영하고 있습니다.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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 hotfix/guide-image-403-fix

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.

Actionable comments posted: 1

🧹 Nitpick comments (1)
.github/ISSUE_TEMPLATE/PULL_REQUEST_TEMPLATE.md (1)

7-13: 템플릿 가독성을 위해 <br><br>는 제거를 권장합니다.

Markdown 템플릿에서는 빈 줄만으로도 충분히 구분돼서, HTML 태그를 빼면 유지보수가 더 깔끔해집니다.

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

In @.github/ISSUE_TEMPLATE/PULL_REQUEST_TEMPLATE.md around lines 7 - 13, Remove
the explicit HTML line breaks `<br><br>` from the template around the "## 😎 해결한
이슈" section and replace them with plain Markdown blank lines (i.e., ensure a
single empty line above and below the header and list content) so the template
uses markdown spacing instead of HTML; search for the `<br><br>` tokens in the
PULL_REQUEST_TEMPLATE.md and delete them, keeping the header "## 😎 해결한 이슈" and
its "- close" list item intact with proper blank-line separation.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/ISSUE_TEMPLATE/PULL_REQUEST_TEMPLATE.md:
- Line 11: Replace the vague placeholder "- close" with a clear issue-close
template such as "- close `#이슈번호`" and add a short inline hint so authors know to
replace "#이슈번호" with the actual issue number (e.g., "- close #<issue-number>").
Update the template line that currently contains "- close" to use this new
default and ensure surrounding PR template sections mention the expected format
(요약 / 😎 해결한 이슈) so contributors follow the `close `#이슈번호`` convention.

---

Nitpick comments:
In @.github/ISSUE_TEMPLATE/PULL_REQUEST_TEMPLATE.md:
- Around line 7-13: Remove the explicit HTML line breaks `<br><br>` from the
template around the "## 😎 해결한 이슈" section and replace them with plain Markdown
blank lines (i.e., ensure a single empty line above and below the header and
list content) so the template uses markdown spacing instead of HTML; search for
the `<br><br>` tokens in the PULL_REQUEST_TEMPLATE.md and delete them, keeping
the header "## 😎 해결한 이슈" and its "- close" list item intact with proper
blank-line separation.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2d8f204c-e448-4a2e-93e8-75d15be89cbf

📥 Commits

Reviewing files that changed from the base of the PR and between 45dc5eb and a454380.

📒 Files selected for processing (2)
  • .github/ISSUE_TEMPLATE/PULL_REQUEST_TEMPLATE.md
  • src/pages/Guide/guideData.ts


## 😎 해결한 이슈

- close
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

이슈 자동 닫힘 포맷 placeholder를 명확히 해주세요.

- close만 있으면 작성자가 형식을 놓치기 쉬워요. - close #이슈번호``로 기본값을 넣어두는 게 좋습니다.

제안 수정안
-- close
+- close `#이슈번호`

Based on learnings: Use PR template sections ✨ 요약 / 😎 해결한 이슈 and issue format close #이슈번호``.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- close
- close `#이슈번호`
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/ISSUE_TEMPLATE/PULL_REQUEST_TEMPLATE.md at line 11, Replace the
vague placeholder "- close" with a clear issue-close template such as "- close
`#이슈번호`" and add a short inline hint so authors know to replace "#이슈번호" with the
actual issue number (e.g., "- close #<issue-number>"). Update the template line
that currently contains "- close" to use this new default and ensure surrounding
PR template sections mention the expected format (요약 / 😎 해결한 이슈) so
contributors follow the `close `#이슈번호`` convention.

@ff1451 ff1451 merged commit e7029e2 into develop Apr 21, 2026
3 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.

1 participant