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

feat: 스터디 생성 페이지 재작성, 구조 재정의 #154

Merged
merged 115 commits into from
May 10, 2024

Conversation

abiriadev
Copy link
Member

@abiriadev abiriadev commented Apr 15, 2024

(WIP)

image

💡 다음 이슈를 해결했어요.

#155

  • 스터디 생성 페이지

  • 스터디 수정 페이지

  • (가능한 한 자세히 작성해 주시면 도움이 됩니다.)


💡 이슈를 처리하면서 추가된 코드가 있어요.



💡 필요한 후속작업이 있어요.

  • 무질서하게 생성된 컴포넌트 정리하기
  • SelectBox의 CSS미적용된 <option> 요소에 들어갈 디자인 작성하고 스타일 적용하기
    • <option>에 스타일을 적용할 방법 논의하기
    • SelectBox의 placeholder에 스타일 적용하기 (6b8cef3 상세 커밋 메세지 참고)
    • atoms.ts 파일을 어디에 놓을 지 정하기. shared? utils?
    • Shared 에 무작위적으로 쌓인 폼 문구, 선택 옵션 등을 일관적으로 정리.


💡 다음 자료를 참고하면 좋아요.

✅ 셀프 체크리스트

  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다. (master/main이 아닙니다.)
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • 변경 후 코드는 컴파일러/브라우저 warning/error 가 발생시키지 않습니다.
  • 변경 후 코드는 기존의 테스트를 통과합니다.
  • 테스트 추가가 필요한지 검토해보았고, 필요한 경우 테스트를 추가했습니다.
  • docs 수정이 필요한지 검토해보았고, 필요한 경우 docs를 수정했습니다.

@abiriadev abiriadev self-assigned this Apr 15, 2024
@abiriadev abiriadev added this to the 스프린트 8 milestone Apr 15, 2024
abiriadev and others added 6 commits April 30, 2024 20:21
- 스터디와 모집 공고 두 경우에서 전부 사용할 수 있음
- 컴포넌트 밖으로 함수를 분리하여 렌덜이 성능 개선 및 기능 분리

(cherry picked from commit de6fd71)
@abiriadev abiriadev marked this pull request as ready for review May 1, 2024 06:16
Copy link
Collaborator

@SungHyun627 SungHyun627 left a comment

Choose a reason for hiding this comment

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

Storybook Setting 및 각 컴포넌트의 초기 Storybook Component 구현하시느라 진짜 수고 많으셨습니다. 페이지를 구성하는 세부 컴포넌트의 Styling 코드까지 분리해서 작성하신 것 같습니다 👍. 추가 기능 구현 시, 기존에 작업했던 것들을 리팩토링하면서 진행할 것 같은데, 작성해주신 공통 컴포넌트 코드 잘 활용하겠습니다!!!. 코드가 지저분했을텐데 ㅜㅜ 코드 이해 + 복잡한 기능 구현 하시느라 고생하셨습니다!!

return <H4 style={{ ...Property[component][type] }}>{children}</H4>;
}
};

Copy link
Collaborator

Choose a reason for hiding this comment

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

Heading 컴포넌트가 깔끔하네요. 다른 페이지에도 적용시키는 과정 진행하겠습니다 👍.

Copy link
Member Author

Choose a reason for hiding this comment

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

@SungHyun627 git blame 보시면 아시겠지만 클레어님이 제작하신 코드입니다

Copy link
Collaborator

Choose a reason for hiding this comment

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

Grid까지 Common 컴포넌트화 하셨네요!! 세심함에 감탄하고 갑니다👍

Copy link
Member Author

Choose a reason for hiding this comment

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

원본은 클레어님이 제작하신 코드입니다...
잘 보시면 Co-Authored-By: hyosin Jang <[email protected]> 가 있습니다.

image

Copy link
Collaborator

Choose a reason for hiding this comment

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

.tsx => .ts로 수정이 필요할 것 같습니다

Copy link
Member Author

@abiriadev abiriadev May 1, 2024

Choose a reason for hiding this comment

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

@SungHyun627
해당 스토리 파일 내에서 JSX 문법을 사용하기 때문에 .tsx 확장자를 사용했습니다. 아마 없으면 vite에서 별도의 JSX loader(= babel 컴파일러) 설정을 해줘야 하는 것으로 알고 있습니다.
혹시 .ts 확장자로도 무사히 작동하는 것을 로컬에서 테스트해주실 수 있을까요?

@@ -0,0 +1,126 @@
import { forwardRef, useMemo } from 'react';
import Select, { ActionMeta, GroupBase, MultiValue, Props, SelectInstance, StylesConfig } from 'react-select';
Copy link
Collaborator

Choose a reason for hiding this comment

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

react-select에서 커스텀할 수 있는 옵션을 많이 제공해주네요👍

Copy link
Member Author

Choose a reason for hiding this comment

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

클레어님이 다 하신 거에요... 흑흑

Copy link
Collaborator

@hyosin-Jang hyosin-Jang left a comment

Choose a reason for hiding this comment

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

abi님이 스토리북 세팅해주신 덕분에 편하게 개발했습니다 🙂
a11y나 remix-react-router 같이 다양한 addon을 제공하는건 처음 알았는데 더 살펴봐야겠어요
고생하셨습니다 👍

.storybook/main.ts Outdated Show resolved Hide resolved
src/Components/Common/Stack/index.tsx Show resolved Hide resolved
src/Pages/Studies/CreateStudy.tsx Outdated Show resolved Hide resolved
src/Pages/Studies/CreateStudy.tsx Outdated Show resolved Hide resolved
src/Types/atoms.ts Show resolved Hide resolved
@abiriadev abiriadev force-pushed the feat/create-study branch from c2d7c9d to 1a2da7f Compare May 7, 2024 11:54
@abiriadev abiriadev merged commit 9654922 into dev May 10, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants