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

프로필 수정 페이지 기본 이미지 세팅 및 버그 수정 #82

Merged
merged 2 commits into from
Aug 15, 2024

Conversation

peter-j0y
Copy link
Member

😎 작업 내용

  • 프로필 이미지 안나오는 버그 앱 로직 수정
  • 프로필 삭제 시 혹은 fallback시 기본 이미지 나오도록 구현

🥲 비고

  • 서버 작업 반영되어야 정상 동작 확인 가능

@peter-j0y peter-j0y self-assigned this Aug 15, 2024
@@ -1,4 +1,4 @@
<?xml version="1.0" encoding="utf-8"?>
<paths>
<external-cache-path name="my_images" path="/"/>
<external-path name="my_images" path="/"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분 때문에 프로필이 안보였던건가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

아뇨 이 부분 때문은 아니고 사진을 촬영하면 사진을 임시로 저장하는 앱의 절대경로를 서버에 그대로 보내주고 있었고 서버에서 값을 받아와도 앱의 임시 경로가 사라지면 asyncImage에서 디코딩이 실패되면서 프로필이 안보였던 상황입니다 그래서 서버에 이미지를 저장해주는 형태로 바꾸는 식으로 해결하려고 request할 때 string이 아니라 multipart로 보내는걸로 바꿨습니다

Copy link
Contributor

Choose a reason for hiding this comment

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

아하.. 그런 이슈였군요 많이 배웠습니다 답변 감사합니다!

@peter-j0y peter-j0y merged commit 824a8d5 into compose/develop Aug 15, 2024
1 check passed
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.

2 participants