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: userId issue in logout mutation #247

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rushikeshacharya
Copy link

@rushikeshacharya rushikeshacharya commented Aug 19, 2024

Content (작업 내용) 📝

  • Issue:
    • The userLogOut function expects a userId of type number as a parameter, but the logout mutation is currently passing the entire User object instead.
  • Fix:
    • Modify the logout mutation to pass userData.id (the user ID) to the userLogOut function instead of the entire User object.

Screenshot for work content (작업 내용 스크린샷) 📸

image

Links (링크) 🔗

  • Issue Links (이슈 링크)

  • API Links (API 스팩 문서)

  • Development Links (개발 문서)

  • Document Links (기획 문서)

  • Design Links (디자인 문서)

Etc (기타 사항) 🔖

Check List (체크 사항) ✅

  • Issue (이슈)
  • Tests (테스트)
  • Ready to be merged (병합 준비 완료)
  • Added myself to contributors table (기여자 추가)

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.

1 participant