Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] 공통 Modal 구현 및 적용, 홈 & 소개 탭 UI 수정사항 반영 #164

Merged
merged 13 commits into from
Dec 2, 2024

Conversation

lydiacho
Copy link
Member

@lydiacho lydiacho commented Nov 29, 2024

✨ 구현 기능 명세

✅ PR Point

변경사항이 매우 많아서 commit 단위로 읽는게 편하실거예요!
혹은 그냥 단순 뷰 수정 작업이었기 때문에 풀받으셔서 뷰만 확인해보셔도 될겁니다!

▶️ 작업한 내용

  • 우측 애니메이션 Modal 공통 컴포넌트로 분리
  • 공통 탭, 탭, 소개 탭에 있는 총 4개의 모달 -> 공통 컴포넌트 적용
  • 지원하기헤더 모달 추가 (공통 컴포로)
  • 소개 탭 UI 변경사항 반영
    • 임원진Info 중 소속 필드는 필수 아니므로 required 및 에러처리 제거
  • 탭 UI 변경사항 반영

    주용오빠가 작업한 메인버튼 컴포를 아예 날려버리기 미안해서 일단 주석처리만 해놓았음

급하게 한 작업이라 코드퀄리티는 보장하지 못하지만...🥲
그래도 공통 분리 해두었으니 추후 수정 작업은 비교적 쉬울 것 같아요

▶️ still TODO

  • 홈 탭 내 최신 소식 추가 후 성공 토스트 메시지 추가해야 함
  • 최신소식 추가 api를 현재 테스트해볼 수 없어서 api 정상화 후 그쪽 레이아웃 한번 더 체크 필요

✅ screenshot

영상엔 없지만 지원하기탭 헤더 모달 누락되어있길래 그것도 추가했습니다!

2024-11-29.10.32.26.mov

@lydiacho lydiacho self-assigned this Nov 29, 2024
Copy link

height bot commented Nov 29, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link

cloudflare-workers-and-pages bot commented Nov 29, 2024

Deploying sopt-admin with  Cloudflare Pages  Cloudflare Pages

Latest commit: 07d4fa1
Status: ✅  Deploy successful!
Preview URL: https://677b46f2.sopt-admin.pages.dev
Branch Preview URL: https://feature-about-view.sopt-admin.pages.dev

View logs

Copy link
Member

@wuzoo wuzoo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ~~!!

Copy link
Member

@eonseok-jeon eonseok-jeon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~~

홈 탭 UI 변경사항 반영
주용오빠가 작업한 메인버튼 컴포를 아예 날려버리기 미안해서 일단 주석처리만 해놓았음

필요없는 코드의 주석은 레거시가 될 수 있으니 미안해 하지 말고 제거해 주세요~~

src/components/org/OrgAdmin/common/Modal/index.tsx Outdated Show resolved Hide resolved
@lydiacho lydiacho merged commit 109930c into dev Dec 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] 공통 Modal 구현 및 적용, 홈 & 소개 탭 UI 수정사항 반영
3 participants