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.

Sign up for GitHub

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

Jump to bottom

[fix] 위니 업로드 / 이미지 크기 5MB 이하로 압축 #180

Merged
merged 6 commits into from
Sep 2, 2023
Merged

[fix] 위니 업로드 / 이미지 크기 5MB 이하로 압축 #180

merged 6 commits into from
Sep 2, 2023

Conversation

leeeha
Copy link
Member

@leeeha leeeha commented Sep 1, 2023 •
edited
Loading

📝 Work Description

  • 1차 릴리즈 마지막 변경 사항 입니다!
  • 포포리 레포 참고해서 이미지 비율은 유지하고 파일 크기는 압축했습니다!
  • 아 그리고 패키지 이름은 처음에 심사 요청했던 어플리케이션과 패키지 이름이 달라야 해서 어쩔 수 없이 바꿨습니닷,,,,
  • 한번 스토어에 올라가고 나서는, 패키지 이름 변경 없이 새로운 버전만 만들면 돼요!
  • 심사 요청 보내고나서 변경 사항 생길 줄은 몰랐는데, 덕분에 이미지 파일 오류도 잡아내고 좋네용 🙂

📣 To Reviewers

  • 이미 aab 파일로 심사 요청 보낸 상태라 여러분이 코드 확인해주면, 메인과 디벨롭으로 모두 머지하도록 할게요!
  • 다들 넘 고생 많았어요 🎉🎉🎉
  • 그리고 더 이상 사용하지 않는 브랜치들은 로컬과 리모트에서 모두 삭제 부탁드립니다!

@leeeha leeeha added fix 💥 버그 및 오류 해결 하은 🐰 labels Sep 1, 2023
@leeeha leeeha added this to the 기능 구현 ✨ milestone Sep 1, 2023
@leeeha leeeha self-assigned this Sep 1, 2023
sxunea
sxunea approved these changes Sep 2, 2023
View reviewed changes
Copy link
Contributor

@sxunea sxunea left a comment

Choose a reason for hiding this comment

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

고생했어요 💯

app/src/main/java/org/go/sopt/winey/util/multipart/UriToRequestBody.kt
fileSize = compressedImage.size.toLong()
}

private fun rotateImageIfRequired(originalBitmap: Bitmap): Bitmap {
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 함수 네이밍 너무 좋네요

app/src/main/java/org/go/sopt/winey/util/multipart/UriToRequestBody.kt
}

override fun contentLength(): Long = size
private fun calcCompressRate(imageSizeMb: Double) =
Copy link
Contributor

Choose a reason for hiding this comment

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

이런식으로 압축할 수 있군요 상수화를 통해서 5MB 이하로도 더 줄일 수 있겠네요
덕분에 알아갑니다 👍 ❤️

Copy link
Member Author

Choose a reason for hiding this comment

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

저도 포포리 코드 덕분에 알게된 방법입니다! :)
이렇게 하니까 파일 크기가 5MB 보다 조금 넘는 경우에는 화질 저하도 심하지 않아서 좋더라구요!! 👍

@leeeha leeeha merged commit 0daeae5 into develop Sep 2, 2023
1 check passed
@leeeha leeeha changed the title [fix] 위니 업로드 / 이미지 파일이 5MB 이상이면 크기 압축 [fix] 위니 업로드 / 이미지 크기 5MB 이하로 압축 Sep 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sxunea sxunea sxunea approved these changes

@Sangwook123 Sangwook123 Awaiting requested review from Sangwook123

Assignees

@leeeha leeeha

Labels
fix 💥 버그 및 오류 해결 하은 🐰
Projects
Winey-AOS
Status: Done
Milestone
기능 구현 ✨
Development

Successfully merging this pull request may close these issues.

[fix] 위니 업로드 / 이미지 크기가 5MB 보다 큰 경우
2 participants
@leeeha @sxunea