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

[Feature] 메뉴 등록 및 수정 기능에서 카메라 촬영 기능 추가 #389

Merged
merged 3 commits into from
Oct 2, 2024

Conversation

hsgo2430
Copy link
Contributor

개요


  • 메뉴 등록 및 수정 시 카메라 촬영 기능 구현

상세 작업 내용


  • ActivityResultContracts.TakePicture()를 사용하여 이미지 촬영 후 등록

작업 결과물

  • 이미지 촬영 후 등록

@hsgo2430 hsgo2430 requested a review from a team as a code owner September 25, 2024 03:38
@hsgo2430 hsgo2430 changed the title 메뉴 등록 및 수정 기능에서 카메라 촬영 기능 추가 [Feature] 메뉴 등록 및 수정 기능에서 카메라 촬영 기능 추가 Sep 25, 2024
Copy link
Contributor

@yunjaena yunjaena left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! comment 확인해주세요

Copy link
Collaborator

@Jokwanhee Jokwanhee left a comment

Choose a reason for hiding this comment

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

오랜만에 코드 리뷰 다네요~ 자주 오겠습니다~ 고생했어요~ ☺️

Comment on lines +46 to +51
fun createImageFile(context: Context): Uri? {
val contentResolver = context.contentResolver
val content = ContentValues().apply {
put(MediaStore.MediaColumns.MIME_TYPE, "image/jpeg")
}
return contentResolver.insert(MediaStore.Images.Media.EXTERNAL_CONTENT_URI, content)
Copy link
Collaborator

Choose a reason for hiding this comment

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

File 유틸 클래스 만들어서 추상화해도 괜찮을 것 같은데 같은데, 어떠시오? 🤠

Copy link
Contributor Author

Choose a reason for hiding this comment

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

추상화.... 한번 만들어볼게요!

Copy link
Collaborator

@Jokwanhee Jokwanhee left a comment

Choose a reason for hiding this comment

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

리뷰 코멘트가 학습 코멘트라 미리 어프로브 해놓을게요~ 고생많았다요~ ☺️

Copy link
Contributor

@yunjaena yunjaena left a comment

Choose a reason for hiding this comment

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

vi

Copy link
Contributor

@yunjaena yunjaena left a comment

Choose a reason for hiding this comment

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

LGTM 수고하셨습니다.

@hsgo2430 hsgo2430 merged commit 3bb5b03 into develop Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants