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

fix: "undefined" is not valid JSON 이슈 #200

Merged
merged 8 commits into from
Nov 22, 2023
Merged

Conversation

bbearcookie
Copy link
Member

🧩 이슈 번호

✅ 작업 사항

  • JSON.stringify(), JSON.parse() 에 대한 예외 처리 로직이 없어서 크래쉬가 나는 문제를 해결했어요.
  • 로그인 응답시 memberId 가 아니라, Id 로 오고 있어서 프로퍼티 명을 교체했어요.

👩‍💻 공유 포인트 및 논의 사항

image

@bbearcookie bbearcookie added ☁️ FE 프론트 레포지토리에서의 작업 🐛 fix 버그 수정 작업 labels Nov 22, 2023
@bbearcookie bbearcookie added this to the 4차 스프린트 milestone Nov 22, 2023
@bbearcookie bbearcookie self-assigned this Nov 22, 2023
Copy link

netlify bot commented Nov 22, 2023

Deploy Preview for moabam-storybook ready!

Name Link
🔨 Latest commit 52c0357
🔍 Latest deploy log https://app.netlify.com/sites/moabam-storybook/deploys/655dbc39fd97aa0008d5c241
😎 Deploy Preview https://deploy-preview-200--moabam-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@chasj0326 chasj0326 left a comment

Choose a reason for hiding this comment

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

확인 완료 !

@bbearcookie bbearcookie merged commit 86538b1 into main Nov 22, 2023
@bbearcookie bbearcookie deleted the fix/#199/json-crash branch November 22, 2023 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
☁️ FE 프론트 레포지토리에서의 작업 🐛 fix 버그 수정 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"undefined" is not valid JSON 이슈
2 participants