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] 잘못된 이미지 url일 때 엑스박스가 표기되는 문제 해결 #875

Merged
merged 7 commits into from
Dec 27, 2024

Conversation

dmdgpdi
Copy link
Collaborator

@dmdgpdi dmdgpdi commented Dec 26, 2024

What is this PR? 🔍

기능

  • 이미지가 엑스박스가 뜨던 이유는 존재하지 않는 자원을 찾는 url 혹은 잘못된 url이 존재하기 때문이었습니다. 따라서 해당 url을 불러오는데 실패하고 엑박이 떴었습니다. 따라서 이미지가 error일 때는 default image를 보여줘야 했습니다.

잘못된 url로 이미지를 불려오려다 실패하는 네트워크 통신 예시
K-022

Changes 📝

  • useImageLoader라는 훅으로 해당 이미지의 상태가 loading 중인지 loaded 됐는지, 에러인지 확인하고 렌더링을 하게 변경했습니다.

ScreenShot 📷

이번 PR에서의 변경점

Before After
K-018 K-019
Before After
K-020 K-021

Precaution

@dmdgpdi dmdgpdi added frontend 프론트 관련 fix 🔥 bug 수정을 포함한 변경 사항 labels Dec 26, 2024
@dmdgpdi dmdgpdi requested a review from obvoso December 26, 2024 14:03
@dmdgpdi dmdgpdi self-assigned this Dec 26, 2024
@dmdgpdi dmdgpdi changed the title [FIX] 못된 이미지 url일 때 엑스박스가 표기되는 문제 해결 [FIX] 잘못된 이미지 url일 때 엑스박스가 표기되는 문제 해결 Dec 26, 2024
@dmdgpdi dmdgpdi added the D-1 label Dec 26, 2024
@sangwonsheep
Copy link
Collaborator

고생하셨습니다! 백엔드에서 파싱해줘야 하는 부분은 주말에 작업 시작하겠습니다!

@dmdgpdi dmdgpdi merged commit a1cde58 into fe-develop Dec 27, 2024
1 check passed
@sangwonsheep sangwonsheep deleted the front/fix/#873 branch December 31, 2024 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-1 fix 🔥 bug 수정을 포함한 변경 사항 frontend 프론트 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants