Skip to content

[CHORE] 사소한 작업들 처리#428

Merged
i-meant-to-be merged 5 commits intodevelopfrom
chore/#427
Mar 1, 2026
Merged

[CHORE] 사소한 작업들 처리#428
i-meant-to-be merged 5 commits intodevelopfrom
chore/#427

Conversation

@i-meant-to-be
Copy link
Contributor

@i-meant-to-be i-meant-to-be commented Feb 26, 2026

🚩 연관 이슈

closed #427

📝 작업 내용

아래 4가지 작업을 진행했어요:

  • npm audit와 GitHub Dependabot에서 확인된 취약점 해결을 위해 패키지 업데이트 진행
  • 써니의 신 기능 소개 팝업 관련 추가 피드백 반영
  • 신 기능 소개 모달이 랜딩 페이지와 테이블 목록 페이지에서만 보이도록 수정
  • 린팅 경고 해결

🏞️ 스크린샷 (선택)

없음

🗣️ 리뷰 요구사항 (선택)

없음

Summary by CodeRabbit

  • 버그 수정

    • 업데이트 모달의 레이아웃 최적화로 콘텐츠 높이와 간격 정렬 개선
    • 반응형 간격 조정으로 다양한 화면에서 일관된 표시 유지
  • 신규 기능

    • 일부 페이지에 업데이트 모달 표시 추가
  • 업데이트

    • 최신 패치 노트 버전 및 관련 이미지가 갱신됨

@i-meant-to-be i-meant-to-be self-assigned this Feb 26, 2026
@i-meant-to-be i-meant-to-be added chore 파일 내부가 아닌 파일 자체에 대한 변경 (디렉토리 이동, 파일 이름 변경, 파일 삭제 등) design UI 관련 labels Feb 26, 2026
@gemini-code-assist
Copy link

Summary of Changes

Hello @i-meant-to-be, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 Pull Request는 주로 보안 강화 및 새로운 사용자 기능 관련 개선에 중점을 둔 사소한 유지보수 작업을 처리합니다. 식별된 취약점을 완화하기 위한 필수 패키지 종속성 업데이트와 새로운 기능 소개 팝업을 개선하기 위한 피드백 반영이 포함되어 있습니다.

Highlights

  • 패키지 업데이트: npm audit 및 GitHub Dependabot에서 발견된 취약점 해결을 위해 여러 패키지 종속성을 업데이트했습니다.
  • 신 기능 소개 팝업 개선: 써니의 신 기능 소개 팝업과 관련된 추가 피드백을 반영하여 UI/UX를 개선했습니다.
Changelog
  • package-lock.json
    • minimatch, ajv, axios, brace-expansion, rollup 및 해당 플랫폼별 바이너리를 포함한 다양한 패키지 버전을 업데이트했습니다.
    • @isaacs/balanced-match 및 @isaacs/brace-expansion 항목을 제거했습니다.
  • src/components/UpdateModal/UpdateModal.tsx
    • UpdateModal 컴포넌트 내 메인 콘텐츠 영역의 높이 및 간격 스타일을 조정했습니다.
  • src/constants/patch_note.ts
    • 새로운 기능 소개와 관련된 최신 패치 노트의 이미지 가져오기 및 버전 번호를 업데이트했습니다.
Activity
  • 현재까지 Pull Request에 대한 인간 활동(댓글, 검토 등)은 없습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@coderabbitai
Copy link

coderabbitai bot commented Feb 26, 2026

Walkthrough

UpdateModal의 레이아웃(높이·간격) 조정, 패치 노트 상수(LATEST_PATCH_NOTE)와 이미지 참조 버전 갱신, 그리고 UpdateModalWrapper의 렌더 위치 변경(일부 페이지에 추가, 라우트 레벨에서는 제거) 등 UI/구성 관련 변경입니다. 제어 흐름의 로직 변경은 없습니다.

Changes

Cohort / File(s) Summary
UpdateModal 레이아웃
src/components/UpdateModal/UpdateModal.tsx
메인 콘텐츠 컨테이너 높이 h-[60%]h-[59.5%]로 조정; 간격 gap-[2%]gap-[clamp(36px,2.75vw,44px)]로 변경(레이아웃/스타일 업데이트).
패치 노트 상수 변경
src/constants/patch_note.ts
패치 이미지 참조를 0001.png0002.png로 변경하고 LATEST_PATCH_NOTE.version'0001''0002'로 업데이트(데이터 값 변경).
UpdateModalWrapper 위치 변경
src/page/LandingPage/LandingPage.tsx, src/page/TableListPage/TableListPage.tsx, src/routes/routes.tsx
UpdateModalWrapper를 LandingPage와 TableListPage에는 추가로 렌더링하도록 삽입했으며, routes 레벨에서는 해당 Wrapper import/사용을 제거하여 전역 렌더링에서는 제외되도록 조정(마운트 위치 변경).
사소한 JSX/포맷 변경
src/page/DebateVotePage/DebateVotePage.tsx, src/page/TableSharingPage/TableSharingPage.tsx
JSX prop/네비게이트 호출의 포맷(줄바꿈) 변경으로 가독성 조정, 기능·동작에는 변화 없음.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • jaeml06
  • useon

Poem

🐰✨ 레이아웃을 살며시 다듬고, 버전도 하나 올렸네,
창문 사이로 패치 이미지 반짝, 새 소식 전하네.
작은 픽셀 하나도 춤추듯 맞춰주며,
토끼는 당근 들고 응원하네 — 홧팅! 🥕🎉

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 20.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive PR 제목이 변경 사항의 핵심을 나타내지 못함. '[CHORE] 사소한 작업들 처리'는 매우 추상적이며 구체적인 변경 내용을 반영하지 않음. 구체적인 변경 내용을 포함하도록 제목을 수정하세요. 예: '[CHORE] 신 기능 소개 모달 노출 범위 제한 및 패치 노트 업데이트'
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed 연관 이슈 #427의 코딩 요구사항(신 기능 소개 모달 노출 범위 제한, 패치 노트 업데이트)이 모두 구현되었음.
Out of Scope Changes check ✅ Passed 모든 변경사항이 연관 이슈 #427의 목표(신 기능 소개 모달 페이지별 제한, 패치 노트 업데이트)와 관련이 있음.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/#427

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.

@github-actions
Copy link

github-actions bot commented Feb 26, 2026

🚀 Preview 배포 완료!

환경 URL
Preview 열기
API Dev 환경

PR이 닫히면 자동으로 정리됩니다.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

안녕하세요. PR 리뷰를 맡게 되었습니다.
이번 PR은 npm audit 및 Dependabot에서 발견된 취약점 해결을 위한 패키지 업데이트와 신규 기능 소개 팝업에 대한 피드백 반영을 포함하고 있네요.

전반적으로 변경 사항들은 좋아 보입니다. 패키지 업데이트를 통해 보안을 강화하고, 팝업 UI를 개선한 점이 인상적입니다.

한 가지 유지보수성 향상을 위한 제안을 src/constants/patch_note.ts 파일에 남겼습니다. 패치 노트 버전과 이미지 경로를 한 곳에서 관리하도록 구조를 개선하면 앞으로 업데이트가 더 편리하고 안전해질 것입니다.

리뷰 확인 부탁드립니다!

Copy link
Contributor

@useon useon left a comment

Choose a reason for hiding this comment

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

의도한 화면에서만 뜨는 것 확인했습니다 !! 감사합니다 !!!!

@i-meant-to-be i-meant-to-be merged commit 63ea82d into develop Mar 1, 2026
3 checks passed
@i-meant-to-be i-meant-to-be deleted the chore/#427 branch March 1, 2026 14:02
@github-actions
Copy link

github-actions bot commented Mar 1, 2026

🧹 Preview 배포가 정리되었습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 파일 내부가 아닌 파일 자체에 대한 변경 (디렉토리 이동, 파일 이름 변경, 파일 삭제 등) design UI 관련

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[CHORE] 사소한 작업들 처리

2 participants