Skip to content

Merge To main: 7월 5주차#467

Merged
MrMirror21 merged 20 commits into
mainfrom
dev
Aug 1, 2025
Merged

Merge To main: 7월 5주차#467
MrMirror21 merged 20 commits into
mainfrom
dev

Conversation

@MrMirror21

@MrMirror21 MrMirror21 commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

Related issue 🛠

Work Description ✏️

Uncompleted Tasks 😅

To Reviewers 📢

Summary by CodeRabbit

  • 신규 기능

    • 직원 이력서 카드, 북마크, 빈 상태 컴포넌트 추가
    • 홈 화면에 커리어 카드, 이력서 카드, 인사말, 상단 섹션 등 다양한 컴포넌트 추가
    • 비자 정보 포매팅 및 표시 방식 개선
    • 커리어 리스트 및 게스트용 커리어 리스트 조회 훅 추가
  • 기능 개선

    • 홈 배너를 캐러셀로 교체
    • 홈 화면에서 사용자 유형(고용주/일반)에 따라 맞춤형 콘텐츠 및 네비게이션 제공
    • 홈 메뉴 및 각종 카드 UI/UX 개선
    • 검색 필터 포매팅 로직 및 타입 구조 개선
  • 버그 수정

    • 지역 필터에서 값이 없을 때 예외 처리 추가
  • 스타일

    • 여러 컴포넌트의 색상, 간격, 정렬 등 스타일 세부 조정
  • 기타

    • Google Analytics를 Google Tag Manager로 교체, Hotjar 추적 스크립트 추가
    • 번역 상수에 비자 관련 추가 문구(‘외’, ‘+’) 포함
    • 타입 정의 및 내비게이션 상태 처리 개선

ohofront and others added 20 commits June 24, 2025 20:49
feat: replace gtag.js with GTM script and add Hotjar tracking
[Feat/464] GTM 및 Hotjar 트래킹 코드 추가
- 기존 page 최상위에서 일괄적으로 px-4를 가지고 있었지만 횡스크롤 형 ui들은 px-4에 의해 잘리면 안되므로 해당 ui가 자체적으로 padding을 가지도록 변경함
- 사용자 유형에 따라 퀵메뉴가 존재하지 않음에도 그 container의 padding이 존재, 화면의 불필요한 공백 생성하고 있어 이를 제거함.
- HomePage 내 UI 레이아웃 변경에 따라 인사 문구 등을 출력하는 HomeGreetingSection, 이력서 완성도를 보여주는ResumeHelperBanner, 사용자 유형별 퀵메뉴를 제공하는 HomeMenu를 묶어 HomeTopSection 컴포넌트로 제공
- 기존 광고 배너 캐러셀을 제공했던 HomeBanner를 HomeBannerCarousel로 변경, HomePage에서 단독 호출
- 사용자 유형별로 공고/커리어 카드 내 요구 비자 리스트 문구를 출력하는 로직을 커스텀 훅으로 분리
- 로직 내 삼항 연산자 제거
- 로직과 ui 로직 상 중복되는 부분이 많은 홈/인재 검색에 사용되는 HomeEmployerEmployeeCardList와 EmployerEmployeeCardList에 사용되는 코드를 EmployeeCard로 통합했습니다.
- 커리어 리스트 요청시 기본 카테고리값 설정
- 커리어 api 훅 주석 수정
- 불필요한 store 조작 코드 제거
Feat/413 유저/기업 홈 수정
@MrMirror21 MrMirror21 added this to the 7월 4주차 스프린트 milestone Aug 1, 2025
@MrMirror21 MrMirror21 requested a review from naarang August 1, 2025 06:23
@MrMirror21 MrMirror21 self-assigned this Aug 1, 2025
@MrMirror21 MrMirror21 added 🐞 BugFix Something isn't working ✨ Feature New feature or request 🎨 Html&CSS Markup & Styles labels Aug 1, 2025
@MrMirror21 MrMirror21 added this to Giggle Aug 1, 2025
@MrMirror21 MrMirror21 added the 📭 API Server API communication label Aug 1, 2025
@coderabbitai

coderabbitai Bot commented Aug 1, 2025

Copy link
Copy Markdown

Warning

Rate limit exceeded

@MrMirror21 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 48 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between e43242a and 913a5dc.

⛔ Files ignored due to path filters (7)
  • src/assets/icons/Home/AppliedIcon.svg is excluded by !**/*.svg
  • src/assets/icons/Home/ApplyIcon.svg is excluded by !**/*.svg
  • src/assets/icons/Home/MatchesIcon.svg is excluded by !**/*.svg
  • src/assets/icons/Home/RecommendPostIcon.svg is excluded by !**/*.svg
  • src/assets/icons/Home/ResumeIcon.svg is excluded by !**/*.svg
  • src/assets/icons/Home/SavedIcon.svg is excluded by !**/*.svg
  • src/assets/icons/Home/SavedPostIcon.svg is excluded by !**/*.svg
📒 Files selected for processing (28)
  • index.html (1 hunks)
  • src/components/Common/EmployeeCard/EmployeeCard.tsx (1 hunks)
  • src/components/Common/EmployeeCard/EmployeeCardBookmark.tsx (1 hunks)
  • src/components/Common/EmployeeCard/EmployeeCardEmptyState.tsx (1 hunks)
  • src/components/Common/EmployeeCard/index.ts (1 hunks)
  • src/components/Employer/EmployeeSearch/EmployerEmployeeCardList.tsx (2 hunks)
  • src/components/Home/HomeBannerCarousel.tsx (4 hunks)
  • src/components/Home/HomeCareerPostCard.tsx (1 hunks)
  • src/components/Home/HomeCareerPostingList.tsx (1 hunks)
  • src/components/Home/HomeEmployeeCardList.tsx (1 hunks)
  • src/components/Home/HomeGreetingSection.tsx (1 hunks)
  • src/components/Home/HomeJobPostingList.tsx (1 hunks)
  • src/components/Home/HomeJobPostingSection.tsx (1 hunks)
  • src/components/Home/HomeMenu.tsx (3 hunks)
  • src/components/Home/HomePostCard.tsx (3 hunks)
  • src/components/Home/HomeTopSection.tsx (1 hunks)
  • src/components/PostSearch/CareerCardList.tsx (2 hunks)
  • src/components/PostSearch/CareerSearchSection.tsx (3 hunks)
  • src/components/PostSearch/PostSearchFilterList.tsx (2 hunks)
  • src/constants/translation.ts (1 hunks)
  • src/hooks/api/useCareer.ts (1 hunks)
  • src/hooks/useEmployeeCard.ts (1 hunks)
  • src/hooks/useFormattedVisa.ts (1 hunks)
  • src/pages/Home/HomePage.tsx (2 hunks)
  • src/pages/PostSearch/PostSearchPage.tsx (3 hunks)
  • src/pages/Resume/ScrappedJobPostsPage.tsx (3 hunks)
  • src/types/api/career.ts (1 hunks)
  • src/utils/formatSearchFilter.ts (1 hunks)
📝 Walkthrough

Walkthrough

공통 EmployeeCard 및 관련 서브 컴포넌트가 신설되어 재사용성을 높이고, 홈 및 검색 관련 주요 섹션이 사용자 유형에 따라 트렌딩/북마크 리스트, 커리어, 이력서 등으로 리팩토링되었습니다. 또한, 구글 애널리틱스가 GTM 및 Hotjar로 대체되었으며, 검색 필터 포맷터와 비자 표기 유틸리티가 도입되었습니다.

Changes

Cohort / File(s) Change Summary
공통 EmployeeCard 컴포넌트 및 유틸리티
src/components/Common/EmployeeCard/EmployeeCard.tsx, .../EmployeeCardBookmark.tsx, .../EmployeeCardEmptyState.tsx, .../EmployeeCard/index.ts, src/hooks/useEmployeeCard.ts, src/hooks/useFormattedVisa.ts
EmployeeCard, Bookmark, EmptyState 등 공통 이력서 카드 UI 및 훅 신설, 북마크/상세 이동 핸들러 분리, 비자 표기 포맷터 도입
홈 주요 섹션 및 카드 컴포넌트 리팩토링
src/components/Home/HomeBannerCarousel.tsx, .../HomeCareerPostCard.tsx, .../HomeCareerPostingList.tsx, .../HomeEmployeeCardList.tsx, .../HomeGreetingSection.tsx, .../HomeJobPostingList.tsx, .../HomeJobPostingSection.tsx, .../HomeMenu.tsx, .../HomePostCard.tsx, .../HomeTopSection.tsx
홈 배너, 인사, 메뉴, 트렌딩/북마크/커리어/이력서 카드 및 리스트 컴포넌트 신설 및 분리, 사용자 유형별 분기, UI/네비게이션 개선
검색 및 리스트 필터/포맷터 개선
src/components/PostSearch/CareerCardList.tsx, .../CareerSearchSection.tsx, .../PostSearchFilterList.tsx, src/utils/formatSearchFilter.ts
커리어/이력서/공고 검색 필터 포맷터 함수 분리 및 타입 강화, CareerCardList에 타이틀/버튼 추가, 게스트/로그인 분기 명확화
API 훅 및 타입 확장
src/hooks/api/useCareer.ts, src/types/api/career.ts
커리어 리스트(게스트/유저) API 훅 추가, CareerListItemType에 이미지 URL 속성 추가
페이지/섹션 통합 및 초기 상태/네비게이션 개선
src/pages/Home/HomePage.tsx, .../PostSearch/PostSearchPage.tsx, .../Resume/ScrappedJobPostsPage.tsx
홈/검색/스크랩 페이지에서 신규 컴포넌트 적용, 검색 옵션 및 탭 초기화 방식 개선, 커리어 정렬·카테고리 파라미터 변경
트래킹 및 번역 상수
index.html, src/constants/translation.ts
구글 애널리틱스 → GTM/Hotjar로 변경, 비자 추가 표기 번역 상수 추가

Sequence Diagram(s)

sequenceDiagram
actor User
participant HomePage
participant HomeTopSection
participant HomePostSection
participant EmployeeCard
participant useEmployeeCard

User->>HomePage: 페이지 진입
HomePage->>HomeTopSection: 렌더링
HomePage->>HomeBannerCarousel: 렌더링
HomePage->>HomePostSection: 렌더링
HomePostSection->>EmployeeCard: 트렌딩/북마크/이력서 리스트 렌더링
EmployeeCard->>useEmployeeCard: 북마크 클릭/상세 이동 핸들러 호출
useEmployeeCard-->>EmployeeCard: 북마크 토글/상세 페이지 이동
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~40 minutes

Assessment against linked issues

Objective Addressed Explanation
트렌딩 잡 리스트(스크랩 수 기준 인기) 및 See More 클릭 시 검색 - 공고/popular 활성화 (#413)
트렌딩 커리어 리스트(스크랩 수 기준) 및 See More 클릭 시 검색 - 커리어/popular 활성화 (#413)
북마크 잡(공고) 및 See More 클릭 시 스크랩 - 공고 활성화 (#413)
북마크 커리어 및 See More 클릭 시 스크랩 - 커리어 활성화 (#413)

Assessment against linked issues: Out-of-scope changes

Code Change Explanation
구글 애널리틱스 → GTM/Hotjar 스크립트 변경 (index.html) 트래킹 코드 변경은 홈/검색/북마크 UI/로직 개선 요구와 직접적 관련이 없음.

Possibly related PRs

Suggested labels

⚙️ Setting

Suggested reviewers

  • ohofront
✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai generate unit tests to generate unit tests for this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@MrMirror21 MrMirror21 merged commit f6468be into main Aug 1, 2025
2 of 3 checks passed
@github-project-automation github-project-automation Bot moved this to Done in Giggle Aug 1, 2025
@github-actions

github-actions Bot commented Aug 1, 2025

Copy link
Copy Markdown

✅ Vitest Test Results

**Status:** ALL TESTS PASSED
**Runtime:** 2.41s

### 📊 Test Summary
```
Test Files: 22 files
Total:      177 tests
Passed:     177 tests
Failed:     0 tests
Skipped:    0 tests
```

📉 Coverage Warnings

⚠️ Lines coverage is below 80% (0%)
⚠️ Functions coverage is below 80% (0%)
⚠️ Branches coverage is below 80% (0%)
⚠️ Statements coverage is below 80% (0%)

🎉 All tests passed!

Great job! All 177 tests are working correctly across 22 test files.

🔗 Quick Actions

  • [🧪 Run tests locally](pnpm test)
  • [📊 Run coverage locally](pnpm test:coverage)
  • [🎨 Open Vitest UI](pnpm test:ui)
  • ⚙️ View workflow logs

🤖 Generated by GitHub Actions at 2025-08-01T06:23:40.608Z

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

Labels

📭 API Server API communication 🐞 BugFix Something isn't working ✨ Feature New feature or request 🎨 Html&CSS Markup & Styles

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

✅ Task Request - 유저/기업 홈 수정

3 participants