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

[TNT-117] 이미지 피커 구현 #31

Merged
merged 4 commits into from
Jan 24, 2025
Merged

[TNT-117] 이미지 피커 구현 #31

merged 4 commits into from
Jan 24, 2025

Conversation

hoyahozz
Copy link
Member

📝 작업 내용


  • 저장소 접근 권한에 따라 사용자가 접근 허용한 이미지만을 선택할 수 있도록 초기 구현을 구상하였으나, 이미지를 '선택만' 할 수 있어도 기획 요구사항이 충족되기 때문에 Android Photo Picker 를 사용하였습니다.

  • 서버측 (@ymkim97) 과의 합의에 따라 이미지의 용량이 10MB를 넘어가지 않도록 구현하였습니다.
    • 만약 이미지 용량이 10MB를 넘을 경우, 자체적으로 리사이징을 진행합니다.

📸 실행 화면

picker.mp4

🙆🏻 리뷰 요청 사항

  • NONE

👀 레퍼런스

@github-actions github-actions bot requested a review from SeonJeongk January 24, 2025 10:25
@hoyahozz hoyahozz added ✨ Feat 기능 구현 🌲 정호 김씨 집안 큰행님 김정호 labels Jan 24, 2025
Copy link
Contributor

@SeonJeongk SeonJeongk left a comment

Choose a reason for hiding this comment

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

고생하셨습니당~👍

) : Transformation {
override val cacheKey: String = "resize_to_max_size_$maxSizeInBytes"

override suspend fun transform(input: Bitmap, size: Size): Bitmap {
Copy link
Contributor

Choose a reason for hiding this comment

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

여기서 size는 무슨 용도인가요??

Copy link
Member Author

@hoyahozz hoyahozz Jan 24, 2025

Choose a reason for hiding this comment

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

오버라이딩 해야해서 매개변수로 들어가 있기는 한데, 지금은 사용하고 있지 않아요!

Size 는 이미지의 사이즈를 지정하는 용도로 사용됩니다~

현재 파라미터로는 지금 설정된 이미지 사이즈가 들어올거에요~

Copy link
Contributor

Choose a reason for hiding this comment

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

아항 넵 확인했습니다!

Comment on lines +52 to +56
val pickMediaLauncher = rememberLauncherForActivityResult(PickVisualMedia()) { uri ->
if (uri != null) {
profileImage = uri
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@hoyahozz hoyahozz merged commit d5ff9a0 into develop Jan 24, 2025
12 checks passed
@hoyahozz hoyahozz deleted the feature/TNT-117 branch January 24, 2025 11:59
@ymkim97
Copy link

ymkim97 commented Jan 24, 2025

감사합니다~~~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 기능 구현 🌲 정호 김씨 집안 큰행님 김정호
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TNT-117] 이미지 피커 구현
3 participants