Skip to content

Conversation

@KingNono1030
Copy link
Contributor

@KingNono1030 KingNono1030 commented Dec 4, 2024

📌 PR 템플릿

🏷️ PR 타입 (PR Type)

아래 해당 사항에 체크해 주세요.

  • 🐛 버그 수정 (Bugfix)
  • ✨ 기능 개발 (Feature)
  • 🎨 코드 스타일 변경 (Code style update) - 포매팅, 로컬 변수 등
  • ♻️ 리팩토링 (Refactoring) - 기능 변화 없음, API 변경 없음
  • 🛠️ 빌드 관련 변경 (Build related changes)
  • 📝 문서 내용 변경 (Documentation)
  • 🔄 기타 (Other) - 설명 작성

📝 요약 (Summary)

PR의 목적과 간단한 설명을 적어주세요.

푸터 개발 완료했습니다 !


🔍 상세 내용 (Describe your changes)

변경 사항을 구체적으로 작성해 주세요.

  • 레이아웃만 신경 쓰고 나머지는 기존의 만든 컴포넌트의 배치 뿐이라 까다롭지 않았습니다.

🔗 관련 이슈 또는 링크 (Issue Number or Link)

이슈 번호나 관련 링크가 있다면 추가해 주세요.


✅ 체크리스트 (Checklist)

PR 작성 시 아래 사항들을 점검해 주세요.

  • 빌드가 성공적으로 되었나요?
  • 코드에 주석을 추가했나요?
  • 모든 테스트가 통과했나요?
  • 관련 문서가 업데이트되었나요?

📸 스크린샷 (선택 사항)

변경 사항이 UI와 관련이 있다면 스크린샷을 추가해 주세요.

image


📝 기타 사항

PR과 관련된 기타 사항이 있다면 적어주세요.

  • 구분선 역할을 하는 divider 컴포넌트의 필요성
  • Tabs 컴포넌트 -> 그냥 Grid 컴포넌트로 대체 가능할 듯

@KingNono1030 KingNono1030 self-assigned this Dec 4, 2024
@KingNono1030 KingNono1030 linked an issue Dec 4, 2024 that may be closed by this pull request
3 tasks
@KingNono1030 KingNono1030 marked this pull request as ready for review December 4, 2024 14:34
Copy link
Contributor

@yellowjang yellowjang left a comment

Choose a reason for hiding this comment

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

혹시 팀미팅에서 말씀하신 구분선 이 적용된건가요 ??

>
(주)DevForDev
<div className='h-14 w-1 bg-gray-200' />
대표 : 고용빈
Copy link
Contributor

Choose a reason for hiding this comment

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

대표 : 정윤호 로 바꾸는 건 어떠신가요 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 대표 고용빈은 용납 못하시는건가요

@KingNono1030
Copy link
Contributor Author

혹시 팀미팅에서 말씀하신 구분선 이 적용된건가요 ??

아니요 추후에 구분선 요구하는 컴포넌트 만들면서 (로그인폼) 그때 공통화 다시 하면 좋을거 같아요

@KingNono1030 KingNono1030 merged commit 0749afa into dev Dec 5, 2024
2 of 3 checks passed
@KingNono1030 KingNono1030 deleted the feat/shared-component-footer branch December 5, 2024 08:45
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.

[FEATURE] [shared 컴포넌트] Footer

3 participants