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/#42] 토스트 컴포넌트 구현 및 연동 #50

Merged
merged 4 commits into from
Feb 7, 2025
Merged

Conversation

lgrin-byte
Copy link
Member

@lgrin-byte lgrin-byte commented Feb 6, 2025

💡 변경사항 & 이슈

토스트 메세지 훅과 리뷰 생성 결과 페이지에 연결했어용

✍️ 관련 설명

애니메이션은 2차 mvp에 추가해야 할 것 같아 안넣었고
텍스트 복사 기능 추가했으

⭐️ Review point

코드 개선점
toast에 더 넣어야 할 속성이 있을지


📷 Demo

스크린샷 2025-02-06 오후 12 08 00


@lgrin-byte lgrin-byte requested a review from sikkzz as a code owner February 6, 2025 03:08
@lgrin-byte lgrin-byte linked an issue Feb 6, 2025 that may be closed by this pull request
1 task
@lgrin-byte lgrin-byte changed the title Feat/#42 [Feat/#42] 토스트 컴포넌트 구현 및 연동 Feb 6, 2025
Copy link

github-actions bot commented Feb 6, 2025

🧷 Storybook: https://6798a432c75b75bf6b81bc26-lvsjufvbwz.chromatic.com/

⏰ Update: 2025년 02월 08일 03시 09분

@sikkzz
Copy link
Collaborator

sikkzz commented Feb 6, 2025

리뷰하기 전에 일단 깃 히스토리가 좀 꼬인거 같아서 내가 바로 잡아서 커밋 넣었어!

코드 바뀐건 없고 이거 히스토리 관리하는 방법은 내가 다음에 만날 때 알려줄게

Copy link
Collaborator

@sikkzz sikkzz left a comment

Choose a reason for hiding this comment

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

음 우선 임시용으로는 써도 될 거 같은데 토스트 메시지도 보통 portal을 활용해서 구현하긴해!

react에서 root DOM과 별개로 동작하게끔 구성하는 경우가 많거든! 전역 store나 context를 사용해서 어디서든 재사용 가능하면서도 별개로 동작하게 구성하는거지

내가 이후에 리팩토링 한번 해볼게! 일단은 내가 리뷰 달은거만 수정해줘~

src/components/ReviewResult/ReviewResult.tsx Outdated Show resolved Hide resolved
src/components/ReviewResult/ReviewResult.tsx Outdated Show resolved Hide resolved
@sikkzz sikkzz merged commit a1a5aaa into develop Feb 7, 2025
2 checks passed
@sikkzz sikkzz deleted the feat/#42 branch February 7, 2025 18:07
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.

[Feature] 토스트 메시지 구현
2 participants