Skip to content

Deploy: develop 브랜치를 main에 병합 - #361

Merged
Sohyunnnn merged 8 commits into
mainfrom
develop
Apr 13, 2026
Merged

Sohyunnnn merged 8 commits into
mainfrom
develop

Conversation

@Sohyunnnn

Copy link
Copy Markdown
Contributor

📌 Summary

main 배포를 위해 develop 브랜치를 병합합니다.

jin-evergreen and others added 8 commits March 23, 2026 14:58
* fix: svg 오류 해결

* fix: 코드리뷰 반영

* fix: 빌드 에러 해결

* fix: 빌드 에러 해결

* fix: ?url 사용

* refactor: icon-urls 엔트리 포인트 분리 및 네이밍 변경

* refactor: 접근성 개선
* feat: 라이브러리 설치

* feat: 이미지 압축 로직 추가

* refactor: 이미지 업로드 중 압축 로직 추가

* feat: option에 webp 변환 추가

* feat: webp 변환 시 파일명 변경

* fix: 동시 업로드 시 압축 상태 조기 해제 버그 수정

* feat: 폼 제출 시 로딩 화면 렌더링

* feat: 기존 파일과 변환 후 파일 크기 비교 로직 추가

* feat: 확장자 없는 파일 대응 로직 추가

* fix: 변수 및 파라미터명 중복 제거

* refactor: 옵션 타입 추가

* fix: async 핸들러 타입 정의 수정

* refactor: 데드코드 제거
* refactor: 알림 구독 로직 수정

* feat: status 관련 API 추가

* refactor: 래빗 리뷰 반영
* refactor: 관객 서비스 PWA 설정 수정

* refactor: 주최 서비스 PWA 설정 수정
@coderabbitai

coderabbitai Bot commented Apr 13, 2026 •

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 인증 상태 확인 기능 추가
    • 이미지 자동 압축 기능 추가
  • 개선 사항

    • 이미지 압축 중 양식 제출 방지
    • 이벤트 편집 워크플로우 개선
    • UI 텍스트 명확성 개선

Walkthrough

인증 상태 확인 기능을 추가하고, 이미지 압축 기능을 구현하며, 모달 로직과 UI를 개선했어요. 또한 아이콘 처리 방식을 컴포넌트 기반에서 URL 기반으로 변경하고 타입 정의를 확장했어요.

Changes

Cohort / File(s) Summary
인증 상태 관리
apps/audience/src/entities/auth/api/auth.ts, apps/audience/src/entities/auth/types/response.ts, apps/audience/src/entities/auth/model/query-options.ts
새로운 인증 상태 조회 API, 응답 타입, React Query 옵션을 추가했어요. 클라이언트에서 인증 상태를 쿼리할 수 있는 기반을 구성했습니다.
인증 상수 및 쿼리키
apps/audience/src/shared/constants/end-point.ts, apps/audience/src/shared/constants/query-key.ts
GET_AUTH_STATUS 끝점과 AUTH_STATUS 쿼리키를 정의했어요.
notice-list 인증 통합
apps/audience/src/pages/notice-list/notice-list.tsx
인증 상태 쿼리를 추가하고, 미인증 사용자는 알럿 클릭 시 인증 페이지로 리다이렉트하도록 변경했어요.
이미지 압축 기능
apps/host/src/shared/libs/image-compress.ts, apps/host/src/features/notice/use-image-upload.ts, apps/host/src/features/notice/use-notice-form.ts, apps/host/src/pages/notice-create/notice-form.tsx
WebP 기반의 이미지 압축 라이브러리를 새로 추가하고, 폼 제출 시 이미지 압축 진행 상태를 추적해서 압축 중에는 제출을 방지하도록 개선했어요.
Flag 아이콘 방식 변경
packages/ads-ui/src/icons/icon-urls.ts, packages/ads-ui/src/icons/index.ts, packages/ads-ui/src/types/svg.d.ts, apps/audience/src/widgets/home/components/flag-button/flag-button.tsx
gradient flag 아이콘을 React 컴포넌트에서 URL 기반 <img> 요소로 변경했어요. ?url 모듈 타입 선언도 추가했습니다.
Event-edit 모달 개선
apps/host/src/pages/event-edit/event-edit.tsx, apps/host/src/features/event-edit/use-event-edit.ts
모달 확인 콜백 구조를 개선하고, 카테고리 삭제 감지 로직을 단순화했어요. 뮤테이션 상태에 따라 UI를 비활성화하고 에러 토스트를 표시하도록 추가했습니다.
컴포넌트 타입 및 UI 개선
apps/host/src/widgets/multi-image-add-button/multi-image-add-button.tsx, apps/host/src/widgets/home/festival-actions/festival-actions.tsx, packages/compositions/src/vite-env.d.ts
onFilesChange 콜백 타입을 async 지원으로 확장했고, festival-actions 삭제 모달의 텍스트와 구조를 개선했어요. Vite 타입 레퍼런스도 추가했습니다.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Suggested labels

🐵 지수, 🐰 소현, 🐱 나은, 🚀 Deploy

Suggested reviewers

  • jin-evergreen
  • jisooooooooooo
  • eunkr82
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 요구사항을 충족합니다. [type] 형식을 따르고 있으며(Deploy), 50자 이내(29자)이고, develop 브랜치를 main에 병합하는 배포 목적을 명확하게 설명합니다.
Description check ✅ Passed PR 설명이 변경 사항과 관련이 있습니다. main 배포를 위한 develop 브랜치 병합이라는 취지가 명확하며, 커밋 메시지 요약에서 구체적인 변경 내용들이 반영되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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.

@Sohyunnnn
Sohyunnnn merged commit a56be65 into main Apr 13, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants