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

스페이스 생성 form 구현 #38

Merged
merged 11 commits into from
Nov 4, 2023
Merged

Conversation

bomi8489
Copy link
Contributor

@bomi8489 bomi8489 commented Nov 2, 2023

📑 이슈 번호

#31

🚧 구현 내용

스페이스 생성 form을 구현하였습니다.
spaceformTest

스페이스 세팅 form을 구현했습니다.
spaceSettingTest

🚨 특이 사항

  1. file input 컴포넌트의 디자인은 임의로 작성했습니다.
  2. @eeseung 드롭다운의 모든 버튼에 type='button'을 추가부탁드립니다. (추가완료)
  3. 스페이스 세팅 form에서 스페이스의 기존 데이터를 불러오는데 카테고리의 데이터를 불러와서 어떻게 default값으로 표시해야할지 몰라서 일단 놔두었습니다. (반영완료)
  4. @dudwns 스페이스 멤버 list의 edit=true 일때 멤버 추가 버튼 컴포넌트가 빠져있습니다. (추가완료)
  5. 새로 발견된 이슈: @eeseung 드롭다운이 모달안에서 쓰일 때 드롭다운을 클릭하면 DropdownItem의 hidden이 바뀌지 않습니다. 드롭다운의 isOpen 상태가 변화를 감지 못하는 것처럼 보입니다.

@bomi8489 bomi8489 added the ✨ Feature 기능 개발 label Nov 2, 2023
@bomi8489 bomi8489 added this to the 1차 스프린트 milestone Nov 2, 2023
@bomi8489 bomi8489 self-assigned this Nov 2, 2023
@bomi8489 bomi8489 linked an issue Nov 2, 2023 that may be closed by this pull request
Copy link
Member

@eeseung eeseung left a comment

Choose a reason for hiding this comment

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

CategoryList에 initialState가 있는 경우가 있었네요..!!😵 첫번째 카테고리가 디폴트여서 컴포넌트 수정이 필요할 것 같은데 이슈를 새로 만들어야 할까요?

Copy link
Member

@eeseung eeseung left a comment

Choose a reason for hiding this comment

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

LGTM 입니다! 👍

@bomi8489 bomi8489 merged commit 5978e75 into develop Nov 4, 2023
1 check passed
@bomi8489 bomi8489 deleted the feature/#31/space-create-form branch November 4, 2023 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

스페이스 생성 form 컴포넌트 구현
2 participants