-
Notifications
You must be signed in to change notification settings - Fork 17
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
토스트 훅으로 분리하고 사용하는 측 코드변경 #423
Conversation
…currency-save 일일 환율 조회 스케줄링 추가
RefreshToken, AccessToken 이용한 로그인 기능 구현
OAuth RedirectURI 잘못 매핑되는 문제 수정
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥🔥
export const useToast = () => { | ||
const setToastList = useSetRecoilState(toastListState); | ||
|
||
const generateToast = (message: string, variant?: ToastType['variant']) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저도 create가 좋은 것 같습니다!
export const useToast = () => { | ||
const setToastList = useSetRecoilState(toastListState); | ||
|
||
const generateToast = (message: string, variant?: ToastType['variant']) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋습니다👍
로컬 cypress 테스트가 실패했습니다. 테스트 결과 자세히 보기 |
📄 Summary
간단하게 만들어봤습니다.
error가 주로 많아서 default를 error로 하는 건 어떠려나요..?
🙋🏻 More
close #419