feat: 메인 Hero 영역을 코어 3기 모집 배너로 임시 대체 - #590
Conversation
64310ab (yarn → pnpm 마이그레이션)에서 갱신되지 않은 pre-commit / commit-msg 훅을 현재 패키지 매니저에 맞게 수정. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
husky 훅과 동일한 마이그레이션 누락 케이스. lint-staged가 스테이징된 JS/TS 파일에 대해 실행하던 'yarn lint:fix'를 'pnpm lint:fix'로 변경. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
디프만 코어 요청으로 인한 임시 작업입니다. - 기존 18기 HeroSection(키링 영상/로고/CTA 조합) 코드는 파일 하단에 주석으로 전량 보존 - 최상단을 코어 3기 모집 배너 이미지(2280x1282) 한 장으로 대체 - 클릭 시 노션 모집 페이지를 새 탭으로 오픈 - 고정 GNB(80px)와 겹치지 않도록 section에 padding-top 적용 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Walkthroughv18 Changes모집 배너 교체
pnpm 명령 전환
Estimated code review effort: 2 (Simple) | ~10 minutes Sequence Diagram(s)sequenceDiagram
participant HeroSection
participant Browser
participant NotionRecruitPage
HeroSection->>Browser: 코어 3기 모집 배너 표시
Browser->>NotionRecruitPage: 새 탭으로 모집 페이지 이동
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/features/Main/sections/v18/HeroSection.tsx`:
- Around line 14-23: Update the link containing the Image in HeroSection so its
accessible name preserves the image alt text, including recruitment dates and
roles, instead of overriding it with the shorter aria-label. Remove the link
aria-label and append an equivalent “opens in a new tab” notice to the Image alt
text while keeping the existing target and rel attributes.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: f6117000-743f-4877-a1d9-887d6499ec4e
⛔ Files ignored due to path filters (1)
public/images/core/recruit-3rd-hero.pngis excluded by!**/*.png
📒 Files selected for processing (4)
.husky/commit-msg.husky/pre-commit.lintstagedrc.jsonsrc/features/Main/sections/v18/HeroSection.tsx
| <a | ||
| href={CORE_RECRUIT_URL} | ||
| target="_blank" | ||
| rel="noopener noreferrer" | ||
| css={linkCss} | ||
| aria-label="디프만 코어 3기 모집 지원하러 가기" | ||
| > | ||
| <Image | ||
| src="/images/core/recruit-3rd-hero.png" | ||
| alt="디프만 코어 3기 모집 - 2026.07.16 ~ 2026.08.02, 프로덕트 디자이너 & 웹 프론트엔드 개발자 모집" |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
링크의 aria-label이 이미지의 접근성 정보를 덮어씁니다.
aria-label 때문에 내부 Image의 alt에 포함된 모집 기간·직군 정보가 링크 이름에서 제외될 수 있으며, 새 탭으로 열린다는 안내도 없습니다. aria-label을 제거하고 alt에 (새 탭에서 열림)을 추가하거나, aria-label에 전체 정보를 명시해 주세요.
수정 예시
- aria-label="디프만 코어 3기 모집 지원하러 가기"
css={linkCss}
>
<Image
src="/images/core/recruit-3rd-hero.png"
- alt="디프만 코어 3기 모집 - 2026.07.16 ~ 2026.08.02, 프로덕트 디자이너 & 웹 프론트엔드 개발자 모집"
+ alt="디프만 코어 3기 모집 - 2026.07.16 ~ 2026.08.02, 프로덕트 디자이너 & 웹 프론트엔드 개발자 모집 (새 탭에서 열림)"📝 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.
| <a | |
| href={CORE_RECRUIT_URL} | |
| target="_blank" | |
| rel="noopener noreferrer" | |
| css={linkCss} | |
| aria-label="디프만 코어 3기 모집 지원하러 가기" | |
| > | |
| <Image | |
| src="/images/core/recruit-3rd-hero.png" | |
| alt="디프만 코어 3기 모집 - 2026.07.16 ~ 2026.08.02, 프로덕트 디자이너 & 웹 프론트엔드 개발자 모집" | |
| <a | |
| href={CORE_RECRUIT_URL} | |
| target="_blank" | |
| rel="noopener noreferrer" | |
| css={linkCss} | |
| > | |
| <Image | |
| src="/images/core/recruit-3rd-hero.png" | |
| alt="디프만 코어 3기 모집 - 2026.07.16 ~ 2026.08.02, 프로덕트 디자이너 & 웹 프론트엔드 개발자 모집 (새 탭에서 열림)" |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/features/Main/sections/v18/HeroSection.tsx` around lines 14 - 23, Update
the link containing the Image in HeroSection so its accessible name preserves
the image alt text, including recruitment dates and roles, instead of overriding
it with the shorter aria-label. Remove the link aria-label and append an
equivalent “opens in a new tab” notice to the Image alt text while keeping the
existing target and rel attributes.
디프만 코어 요청으로 인한 임시 배너의 최신 시안을 반영합니다. '디프만 코어' 문구가 추가된 버전이며, 이미지 규격(2280x1282)은 동일합니다. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 participant
개요
메인 페이지 최상단 Hero 영역을 코어 3기 모집 배너 이미지 한 장으로 임시 대체합니다.
작업 내용
1. Hero 영역 대체 (
src/features/Main/sections/v18/HeroSection.tsx)<a>태그로 대체https://depromeet.notion.site/recruit-3th-opentarget="_blank"+rel="noopener noreferrer"로 새 탭 오픈2. 배너 이미지 추가
public/images/core/recruit-3rd-hero.png(2280 x 1282, 약 16:9)3. 레이아웃
width: 100%+height: auto(단일 이미지 반응형)padding-top을 주어 헤더 영역과 겹치지 않도록 처리4. yarn -> pnpm 잔여 참조 정정 (임시 작업 아님 / 별도 커밋)
64310ab refactor: yarn -> pnpm 변경당시 누락된 husky 훅과 lint-staged 설정을 정정했습니다. 현재 상태에서는 커밋 자체가yarn: not found로 실패합니다..husky/pre-commit:yarn lint-staged->pnpm lint-staged.husky/commit-msg:yarn commitlint --edit->pnpm commitlint --edit.lintstagedrc.json:yarn lint:fix->pnpm lint:fix원복 방법
HeroSection.tsx하단 주석 블록의 기존 코드를 복원하고, 상단 임시 코드와public/images/core/를 제거하면 됩니다.🤖 Generated with Claude Code
Summary by CodeRabbit
새 기능
개선 사항