Skip to content

chore: 도메인을 caquick.com으로 이전 - #81

Merged
chanwoo7 merged 1 commit into
developfrom
chore/domain-caquick-com
Nov 1, 2026
Merged

chanwoo7 merged 1 commit into
developfrom
chore/domain-caquick-com

Conversation

@chanwoo7

@chanwoo7 chanwoo7 commented Nov 1, 2026

Copy link
Copy Markdown
Member

개요

caquick.site(2026-11-29 만료)에서 caquick.com으로 도메인을 옮깁니다. 관리자 웹은 admin.caquick.com, API는 https://api.caquick.com을 씁니다.

  • 이미지 하나에 JS 번들의 API 주소와 nginx CSP가 같이 들어가므로 둘을 한 번에 바꿉니다.
  • develop 머지는 배포가 아니고, 릴리즈(main)는 BE 운영 주소 전환과 같은 시간대에 합니다. 먼저 배포되면 admin.caquick.site ↔ api.caquick.com이 서로 다른 사이트가 되어 refresh 쿠키가 실리지 않습니다.

변경 사항

  • API 주소 기본값: Dockerfile·build-image.yml의 VITE_API_BASE_URL 기본값을 https://api.caquick.com으로 바꿨습니다.
  • CSP: connect-src를 https://api.caquick.com으로 바꿨습니다.
  • 화면 표기: 사이드바의 도메인 표기를 admin.caquick.com으로 바꿨습니다.
  • 문서: README·아키텍처·infra 문서와 주석을 바꾸고, 결정 D18을 추가했습니다(D9 대체).

테스트

  • nginx-config.spec의 CSP 기대값을 새 API 오리진으로 갱신했습니다.
  • pnpm validate 단계(lint·typecheck·codegen:check·knip·test:cov 1165건·build) 통과
  • 빌드 결과물에 api.caquick.com만 들어 있고 api.caquick.site는 없음을 확인했습니다.

플랜 대조

플랜 1단계 관리자 FE 불릿 상태
Dockerfile·build-image.yml의 API 기본값 .com 한 것
nginx CSP connect-src .com 한 것
화면의 도메인 표기 .com 한 것
문서 .com 한 것
머지는 전환 때(릴리즈가 배포) 한 것 — develop 머지만 하고 릴리즈는 전환 때

caquick.site 만료(2026-11-29)에 맞춰 관리자 웹 admin.caquick.com·API api.caquick.com으로 옮긴다. 이미지 하나에 API 주소와 CSP가 같이 들어가므로 한 번에 바꾸고, 릴리즈는 BE 운영 주소 전환과 같은 시간대에 한다.

- VITE_API_BASE_URL 기본값(Dockerfile·build-image.yml), CSP connect-src, 사이드바 표기, 문서·주석
- 결정 D18 추가(D9 대체)
- nginx-config.spec CSP 기대값 갱신
@coderabbitai

coderabbitai Bot commented Nov 1, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-01T00:58:49.401051Z 134d37a PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 96.07% (🎯 96%)
🟰 ±0%
2447 / 2547
🟢 Statements 95.54% (🎯 95%)
🟰 ±0%
2680 / 2805
🟢 Functions 94.29% (🎯 94%)
🟰 ±0%
1008 / 1069
🟢 Branches 88.07% (🎯 88%)
🟰 ±0%
2017 / 2290
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/app/shell/app-shell.tsx 90.9%
🟰 ±0%
100%
🟰 ±0%
75%
🟰 ±0%
90.9%
🟰 ±0%
57
src/shared/api/config.ts 100%
🟰 ±0%
50%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
Generated in workflow #161 for commit 134d37a by the Vitest Coverage Report Action

@chanwoo7
chanwoo7 merged commit 8183af8 into develop Nov 1, 2026
14 checks passed
@chanwoo7
chanwoo7 deleted the chore/domain-caquick-com branch November 1, 2026 01:00
@codecov

codecov Bot commented Nov 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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