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

✨ 미션 선택 Bottom Sheet 연결 #118

Merged
merged 8 commits into from
Dec 22, 2023
Merged

Conversation

sumi-0011
Copy link
Member

🎉 변경 사항

  • 미션 카테고리, 공개 설정 선택 Bottom Sheet를 연결했어요.

🙏 여기는 꼭 봐주세요!

  • 해당 화면에서 사용하는 Bottom Sheet를 한 컴포넌트로 통일할 수 있을 것 같아요! 그런데 급하게 해가지고 중간 발표 이후에 리팩토링 하겠습니다.
  • 카테고리 선택에 아이콘이 있었는데, 생각치도 못해서 생략했어요 >_< (디자이너들이 이정돈 봐주지 않을까여?)

🌄 스크린샷

Dec-23-2023 01-23-06

📚 참고

Copy link

vercel bot commented Dec 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
10mm-client-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 22, 2023 4:43pm

@sumi-0011 sumi-0011 changed the base branch from develop to Design/category-ui December 22, 2023 16:26
@sumi-0011 sumi-0011 changed the base branch from Design/category-ui to develop December 22, 2023 16:27
const useToggle = (initialValue?: boolean): Return => {
const [value, setValue] = useState(initialValue ?? false);

const setTrue = useCallback(() => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

혹시 useCallback으로 하지 않아도 될것 같은데
설정해주신 이유가 궁금해요!

Copy link
Member Author

@sumi-0011 sumi-0011 Dec 22, 2023

Choose a reason for hiding this comment

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

복붙했어요..
뻴까요?

{required && <span className={asterisk}>*</span>}
{required && <span className={asterisk}> *</span>}
Copy link
Member

Choose a reason for hiding this comment

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

이 부분은 오타인거겠죠?? 제가 뷰 구현하다 실수한 부분같아요 죄송합니다ㅠ_ㅠ

@sumi-0011 sumi-0011 merged commit cdd325f into develop Dec 22, 2023
3 checks passed
@sumi-0011 sumi-0011 deleted the feat/mission-bottomsheet branch December 22, 2023 16:44
Copy link
Contributor

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.

3 participants