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] org admin 빈 곳 있을 시 해당 섹션으로 이동 #179

Open
wants to merge 10 commits into
base: refactor-org-register-naming
Choose a base branch
from

Conversation

eonseok-jeon
Copy link
Member

@eonseok-jeon eonseok-jeon commented Dec 26, 2024

✅ PR Point

더 효율적인 방법이 없을까 고민을 많이 했는데 찾지 못했어요
일일이 다 값 비교해보고 빈 값 있으면 해당 섹션으로 이동하게 했어요
홈 탭은 아직 주용이 pr 머지 안 되어서 일단 그대로 뒀어요

일단 현재 페이지에 빈 곳이 있는지 없는지 체크를 먼저하게 했어요
특히 Chip을 이용하는 곳은 빈 곳이 있을 경우 해당 Chip이 선택 되도록 했어요
여러 곳이 비어있다면 그 중 제일 첫 번째 input만 error 처리 되도록 했어요
다 채워져 있다면 공통 -> 홈 -> 소개 -> 지원하기 탭 순으로 체크하게 했습니다

추가로 빈 필드가 있을 시 error toast가 뜨도록 구현했어요
스크린샷 2024-12-27 오후 10 45 35

앞으로 해야할 것

  • 수정하기 탭 input들 에러 상태 UI 처리
  • 홈 탭 input들 validate 추가

@eonseok-jeon eonseok-jeon self-assigned this Dec 26, 2024
Copy link

height bot commented Dec 26, 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 Dec 26, 2024

Deploying sopt-admin with  Cloudflare Pages  Cloudflare Pages

Latest commit: 43a383c
Status: ✅  Deploy successful!
Preview URL: https://2943617c.sopt-admin.pages.dev
Branch Preview URL: https://feature-org-common-fill-yb.sopt-admin.pages.dev

View logs

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] org admin 빈 곳 있을 시 해당 섹션으로 이동
1 participant