Skip to content

fix: 푸터 고객문의 링크 제거 - #134

Merged
evenif99 merged 1 commit into
devfrom
fix/remove-footer-contact-link
Jun 23, 2026
Merged

fix: 푸터 고객문의 링크 제거#134
evenif99 merged 1 commit into
devfrom
fix/remove-footer-contact-link

Conversation

@evenif99

@evenif99 evenif99 commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • HomeServiceInfo 공통 컴포넌트에서 고객문의 항목 제거
  • 서비스 소개, 개인정보 처리방침, 이용약관 3개만 표시되도록 수정

Test plan

  • 홈 화면 푸터에서 고객문의가 사라지고 3개 링크만 표시되는지 확인
  • 서비스 소개 페이지 푸터에서도 동일하게 반영되는지 확인

Summary by CodeRabbit

버그 수정

  • 서비스 정보 영역의 네비게이션에서 비활성 상태로 표시되던 고객문의 링크가 제거되었습니다. 이제 서비스 소개, 개인정보 처리방침, 이용약관 링크만 표시됩니다.

- HomeServiceInfo 공통 컴포넌트에서 고객문의 항목 삭제
- 서비스 소개, 개인정보 처리방침, 이용약관 3개만 표시

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
talky-owl Ready Ready Preview, Comment Jun 23, 2026 8:11am

@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: d34b2c71-05ed-4bf7-8466-ee2307d1cc33

📥 Commits

Reviewing files that changed from the base of the PR and between 89cf4aa and 2cb20cc.

📒 Files selected for processing (1)
  • src/components/home/HomeServiceInfo.tsx
💤 Files with no reviewable changes (1)
  • src/components/home/HomeServiceInfo.tsx

📝 Walkthrough

Walkthrough

HomeServiceInfo.tsxLINKS 배열에서 href: null고객문의 항목을 제거했습니다. 이로 인해 aria-disabled="true" 속성의 비활성 <span>이 더 이상 렌더링되지 않으며, 나머지 3개의 유효한 링크는 그대로 유지됩니다.

Changes

고객문의 링크 제거

Layer / File(s) Summary
LINKS 배열에서 고객문의 항목 제거
src/components/home/HomeServiceInfo.tsx
LINKS 배열에서 href: null고객문의 항목을 제거하여 비활성 <span aria-disabled="true"> 렌더링이 사라지고, 내비게이션은 유효한 3개 링크만 처리함.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 한 줄이 사라졌네, 고객문의여 안녕~
href가 null이라 비활성 신세였지,
aria-disabled span도 이제는 없어졌고,
세 개의 링크만 남아 오늘도 빛나네.
깔끔해진 배열, 토끼도 기쁘다네! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명이 주요 내용과 테스트 계획은 포함하고 있으나, 리포지토리의 필수 템플릿에서 요구되는 대부분의 섹션이 누락되어 있습니다. PR 템플릿의 필수 섹션들(담당 작업 영역, 관련 Issue, 변경사항, 테스트 결과, 작업 범위 확인, 보안/개인정보 확인 등)을 완성하여 제출하시기 바랍니다.
✅ Passed checks (4 passed)
Check name Status Explanation
Title 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/remove-footer-contact-link

Comment @coderabbitai help to get the list of available commands.

@evenif99
evenif99 merged commit 584d593 into dev Jun 23, 2026
3 checks passed
@evenif99
evenif99 deleted the fix/remove-footer-contact-link branch June 23, 2026 08:11
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