-
Notifications
You must be signed in to change notification settings - Fork 1
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
[fix] 나중에 고를래요 누르면 빈 배열 보내게 만들기 #312
[fix] 나중에 고를래요 누르면 빈 배열 보내게 만들기 #312
Conversation
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.
고생하셨습니다~
const [selectedCategories, setSelectedCategories] = useState<number[]>([]); | ||
const [isSettled, setIsSettled] = useState(false); // 버튼 누르고 성공 응답 오기 전에 로딩 스피너 띄우기 |
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.
isSubmitting 으로 하는게 어떨까요?
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.
좋습니다~
onError: async (error) => { | ||
await console.log(error); | ||
}, |
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.
에러면 못 넘어가는군요. 에러 났을 때 잠시 후 다시 시도해주세요 또는 실패했어요. 다음에 다시 선택하시겠어요? 이런식으로 보여줘도 될 것 같아요~ 예 클릭하면 그냥 /로 이동하는 걸로요~!
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.
음 이 행위 자체가 '나중에 고를래요' 버튼을 누르는 것이기 때문에 실패했을 때 다음에 다시 선택하시겠어요? 이걸 보여주는 것 자체가 중복인 느낌이라서 족므 더 고민이네요. 조금 더 생각해봐야할 것 같습니다.
📄 Description of the PR
🔧 What has been changed?
기존에는 나중에 고를래요 누르면 아무 것도 안 보내서 다음 로그인 때 또 login/add 페이지가 뜨는 문제가 발생했습니다.
이를 해결하기 위해 나중에 고를래요 버튼을 누를 때 아무것도 안 보내는 것이 아닌 빈 배열을 보내어 DB에 기록을 남김으로써 로그인을 다시 해도 login/add페이지가 안 뜨게 만들었습니다.
📸 Screenshots / GIFs (if applicable)
✅ Checklist