Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feature/#371 학습자료 상세 조회 에러 처리 #377

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

yeonddori
Copy link
Collaborator

@yeonddori yeonddori commented Jan 12, 2025

관련 이슈

작업 요약

  • 비회원 및 팀원이 아닌 회원이 학습자료 상세 조회 접근 시 모달창으로 접근 불가 알림을 표시하도록 하였습니다.

작업 상세 설명

  • 서버를 이전하게 되면서 제대로 작동하는지 아직 확인을 못 해봤습니다.
  • 로컬에서 임의로 확인해 본 터라, 추후에 확인해 본 후 다시 수정하겠습니다!
    => 확인해본 결과, Application error는 뜨지 않았습니다. 그래도 다들 확인 부탁드립니다!

리뷰 요구 사항

  • 처음에는 에러 처리를 카드 내에서 표시하려 했으나, 디자인적으로 애매할 것 같아 AlertModal을 활용해 표시했습니다. 하지만 이제 생각해보니, 스터디 카드의 진행 상태처럼 작은 Badge 형태로 '로그인 필요', '팀원만 가능' 등의 메시지를 표시하는 것도 괜찮아 보이네요.. 하지만 클릭했을 때 아무 반응이 없으면 또 이상할 것 같기도 하고.. 너무 어렵네요😵‍💫 지금 구현한 모달 알림창 형식은 다들 괜찮으신지 의견 부탁드립니다!

미리 보기

스크린샷 2025-01-12 154523
스크린샷 2025-01-12 161433

@yeonddori yeonddori added the ✨ 기능 개발 새로운 기능을 구현하는 데 필요한 작업 또는 변경 사항 label Jan 12, 2025
@yeonddori yeonddori self-assigned this Jan 12, 2025
Copy link
Collaborator

@llddang llddang left a comment

Choose a reason for hiding this comment

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

image
image

위와 같은 에러가 나고 있습니다.
이유는 서버에서 값을 가져올 수 없어서 인 것 같습니다.
확인 부탁드립니다! :)

@yeonddori yeonddori force-pushed the Feature/#371-학습자료_상세_조회_에러_처리 branch from e88f993 to e167933 Compare February 9, 2025 08:27
…into Feature/#371-학습자료_상세_조회_에러_처리
…into Feature/#371-학습자료_상세_조회_에러_처리
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
✨ 기능 개발 새로운 기능을 구현하는 데 필요한 작업 또는 변경 사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 학습자료 상세 조회 에러 처리
2 participants