Skip to content

fix: 파견학교 지도 초기 줌 조정 - #569

Merged
manNomi merged 1 commit into
mainfrom
fix/university-map-overview-zoom
Jun 26, 2026
Merged

fix: 파견학교 지도 초기 줌 조정#569
manNomi merged 1 commit into
mainfrom
fix/university-map-overview-zoom

Conversation

@manNomi

@manNomi manNomi commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

작업 내용

  • 파견학교 상세 페이지의 Google Embed Map 초기 zoom을 5로 지정했습니다.
  • 기존보다 덜 상세하게 보여서 캠퍼스 주변보다 국가/지역 기준의 위치감을 먼저 확인할 수 있도록 조정했습니다.

검증

  • pnpm --filter @solid-connect/university-web run lint:check
  • pnpm --filter @solid-connect/university-web run typecheck
  • NODE_ENV=production pnpm --filter @solid-connect/university-web run build
  • pre-push hook의 university-web ci:checkbuild
  • university-web 정적 산출물에서 Google Maps iframe zoom=5 반영 확인

참고

  • Google Maps zoom 값은 숫자가 클수록 더 상세하게 확대됩니다. 이번 변경은 사용자가 국가/지역 단위의 위치를 먼저 파악하도록 기존보다 낮은 zoom 값을 사용합니다.

@manNomi
manNomi requested review from enunsnv and wibaek as code owners June 22, 2026 06:26
@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
solid-connect-university-web Ready Ready Preview, Comment Jun 22, 2026 6:27am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jun 22, 2026 6:27am
solid-connection-web Skipped Skipped Jun 22, 2026 6:27am

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 15934f6a-16e1-47ba-8f14-15c5606d90a0

📥 Commits

Reviewing files that changed from the base of the PR and between 05dd9d0 and 427bd86.

📒 Files selected for processing (1)
  • apps/university-web/src/app/university/[homeUniversity]/[id]/_ui/UniversityDetail/_ui/MapSection.tsx

Walkthrough

  1. zoom 상수 추가

    • MapSection.tsx 파일에 SCHOOL_LOCATION_OVERVIEW_ZOOM 상수(값: 5)가 로컬로 새롭게 정의되었습니다.
  2. GoogleEmbedMap zoom prop 변경

    • 기존에 하드코딩되어 있던 zoom 값 대신, 새로 선언된 SCHOOL_LOCATION_OVERVIEW_ZOOM 상수를 GoogleEmbedMapzoom prop에 전달하도록 변경되었습니다.
  3. JSX 포맷 재정리

    • GoogleEmbedMap의 props 작성 방식이 한 줄(single-line) 형태에서 여러 줄(multi-line) 들여쓰기 구조로 재포맷되었습니다.
    • 전달되는 props(width, height, name, zoom, style)의 값 자체는 변경되지 않았습니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경사항의 핵심을 명확하게 전달합니다. 파견학교 지도 초기 줌 조정이라는 주요 변경사항을 간결하고 구체적으로 설명합니다.
Description check ✅ Passed PR 설명이 필수 섹션을 모두 포함하며 충분히 상세합니다. 관련 이슈, 작업 내용, 검증 과정, 참고사항이 모두 명확하게 기술되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/university-map-overview-zoom

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.

@manNomi
manNomi merged commit 10c8135 into main Jun 26, 2026
13 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.

학교 상세페이지 지도를 개선한다

2 participants