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.

Sign up for GitHub

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

Jump to bottom

[ Fix / toastCreate / #178 ] toast 무한 생성 버그 수정 #179

Merged
merged 8 commits into from
Sep 25, 2023
Merged

[ Fix / toastCreate / #178 ] toast 무한 생성 버그 수정 #179

merged 8 commits into from
Sep 25, 2023

Conversation

ljh0608
Copy link
Member

@ljh0608 ljh0608 commented Sep 19, 2023 •
edited
Loading

✨ 해당 이슈 번호 ✨

#178

todo

  • 토스트 창 하나만 띄어지게 하기
  • 토스트 창이 다음 페이지 넘어가는 것을 막기

📌 내가 알게 된 부분

  • toast 를 생성할 땐 id값을 넣어줘야 예상 가능하게 작동한다 (광클해도 한 번만 뜹니다. dismiss같은 속성도 이용가능하구요)

📌 공유하고 싶은 부분

  • toastContainer는 app (route)에 넣는 것이 권장됩니다.
  • toast는 id를 사용하면 연속적으로 마구잡이로 생성되지 않습니다.
  • limit속성은 최대 몇개 띄울 것이냐 인데 id가 없어도 동작은하지만 한개를 계~~속 띄우게되니 id를 사용해야합나다.

📌 질문할 부분

-없음!

📌스크린샷

2023-09-19.7.28.54.mov

@ljh0608 ljh0608 added fix fix 재훈 재훈이의 개발 라벨 labels Sep 19, 2023
@ljh0608 ljh0608 self-assigned this Sep 19, 2023
@vercel
Copy link

vercel bot commented Sep 19, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
asap ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 25, 2023 7:52am

simeunseo
simeunseo approved these changes Sep 19, 2023
View reviewed changes
Copy link
Member

@simeunseo simeunseo left a comment •
edited
Loading

Choose a reason for hiding this comment

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

굳굳!! 근데 제가 토스트 height 수정한게 적용이 안되어있네요 뭔가 브랜치가 .. 머지가 안됐나?? 컨플릭트 해결할 때 유의해야할것 같아요
그리고 PR 타이틀
[기능명/#이슈번호] PR 제목
이런식으로 맞춥시다!! PR 템플릿에 주석으로 저렇게 설명되어있더라구요

@ljh0608 ljh0608 changed the title Fix/toast create/#178 [ 토스트 무한 생성 버그 삭제 / #178 ] Fix/toast create/#178 Sep 19, 2023
@ljh0608 ljh0608 changed the title [ 토스트 무한 생성 버그 삭제 / #178 ] Fix/toast create/#178 [ Fix / toastCreate / #178 ] toast 무한 생성 버그 수정 Sep 19, 2023
eastlaw80
eastlaw80 reviewed Sep 20, 2023
View reviewed changes
Copy link
Contributor

@eastlaw80 eastlaw80 left a comment

Choose a reason for hiding this comment

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

위쪽에서 은서가 말해준 토스트창 수정사항 풀 안당겨진 건 이제 해결된건가요? 그 외에는 좋습니다!

@ljh0608 ljh0608 force-pushed the fix/toastCreate/#178 branch from b8e7a91 to d16ce1a Compare September 25, 2023 07:51
@github-actions github-actions bot added size/M and removed size/S labels Sep 25, 2023
@ljh0608 ljh0608 merged commit 44f7fbd into develop Sep 25, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eastlaw80 eastlaw80 eastlaw80 left review comments

@simeunseo simeunseo simeunseo approved these changes

Assignees

@ljh0608 ljh0608

Labels
fix fix size/M 재훈 재훈이의 개발 라벨
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ljh0608 @simeunseo @eastlaw80 @Chanwoo-Jeong