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: mypage revalidate error 수정 #160

Merged
merged 4 commits into from
Dec 16, 2024
Merged

Conversation

gahyuun
Copy link
Member

@gahyuun gahyuun commented Dec 3, 2024

📌 작업 내용

closed #158

구현 내용 및 작업 했던 내역

  • fetch-ax migration
  • 과목 api에 revalidate 추가

🤔 고민 했던 부분

  • 마이페이지에서 과목 추가를 했을 때, 새로고침하지 않는 이상 과목 테이블에 업데이트 되지 않아서 확인했더니 revalidate tag가 존재하지 않아 추가된 과목이 바로 노출되지 않는 것을 확인했습니다
  • 과목 추가, 삭제시에는 서버 데이터와 동기화가 되어있어야 한다고 생각하기에 revalidateTag를 사용해서 동기화 시켜주었습니다.

@gahyuun gahyuun self-assigned this Dec 3, 2024
Copy link

github-actions bot commented Dec 3, 2024

yougyung
yougyung previously approved these changes Dec 4, 2024
Copy link
Member

@yougyung yougyung left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 👍

@yougyung
Copy link
Member

yougyung commented Dec 4, 2024

과목 삭제의 경우에는 useOptimistic을 통해서 데이터 관리를 의도하신 듯 보였는데, 삭제에서도 revalidate를 적용하신 이유가 궁금해요 !

@gahyuun
Copy link
Member Author

gahyuun commented Dec 15, 2024

과목 삭제의 경우에는 useOptimistic을 통해서 데이터 관리를 의도하신 듯 보였는데, 삭제에서도 revalidate를 적용하신 이유가 궁금해요 !

처음에는 안정적인 데이터 관리를 위해 서버와 동기화를 시켜야 한다는 생각으로, 삭제에서도 revalidate를 적용했습니다! 하지만 revalidate를 시키지 않았을때 동기화 되지 않아서 발생하는 문제 상황이 딱히 생각나지 않았구 애초에 useOptimistic을 한다는 것은 클라이언트에서 데이터를 관리한다는 의미이므로 삭제에서 revalidate를 시킬 필요가 없다고 판단되어 코드 제거했습니다!
좋은 의견 감사합니다 🤗

@gahyuun
Copy link
Member Author

gahyuun commented Dec 16, 2024

참고로 크로마틱 관련한 에러가 발생하는데 이 부분에 대해서는 추후 PR 올리겠습니다!
#166

Copy link
Collaborator

@seonghunYang seonghunYang left a comment

Choose a reason for hiding this comment

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

  • 수고하셨습니다!

@gahyuun gahyuun merged commit c2131cf into main Dec 16, 2024
1 of 3 checks passed
@gahyuun gahyuun deleted the mypage-revalidate-error/#158 branch December 16, 2024 04:54
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.

Mypage 추가된 과목 반영 안되는 문제 해결
3 participants