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] 프로필 수정 API 버그 픽스 - #258 #259

Merged
merged 1 commit into from
Sep 4, 2024
Merged

Conversation

sjk4618
Copy link
Member

@sjk4618 sjk4618 commented Sep 4, 2024

🔥Pull requests

⛳️ 작업한 브랜치

👷 작업한 내용

  • 안드에서 원래 프로필 이미지를 그대로 사용할 경우의 버그 픽스
       // 기본이미지로 변경
        if(isNewImageNull && isDefaultImage) {
            //원래 이미지가 기본 이미지가 아닐 경우
            if(userImage != null) {
                deleteImage(userImage);
            }
            foundUser.setImageUrl(null);

        // 원래 이미지를 그대로 사용하거나, 새로운 이미지로 변경
        } else if(!isDefaultImage) {

            // 아요 : 이게 원래 사진 그대로 사용했거나, 새로운 사진으로 변경
            // 안드 : 원래 이미지에서 새로운 이미지
            if(userImage != null && !isNewImageNull) {
                deleteImage(userImage);
                String newImageUrl = getImageUrl(newImage);
                foundUser.setImageUrl(newImageUrl);
            }

            //안드에서 원래 이미지를 그대로 사용하면 굳이 뭐를 안해도됨
        }

📟 관련 이슈

Copy link
Contributor

@gardening-y gardening-y left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

@sjk4618 sjk4618 merged commit 60fa870 into develop Sep 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 프로필수정 API 이미지 버그 픽스
2 participants