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] #39 카카오톡 공유하기 전송 구현 #80

Merged
merged 16 commits into from
Nov 18, 2024

Conversation

jiwon2724
Copy link
Member

Key Changes

Resolves: #39

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

To Reviewers

  • 이거 좀 이상한데? 싶은거 코멘트 남기세여ㅎㅎ

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 정해진 코딩 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트)

Etc.

  • Android -> IOS만 현재 완료 되었습니다. Android에서 공유 받을 때, intent를 적절하게 사용해야하는데 컴포즈에서 라이프사이클 부분 찾아보고 적용할게요.
  • 내일부터 새로운 피처와 공유하기랑 병행해서 진행할게용

@jiwon2724 jiwon2724 requested a review from l5x5l November 17, 2024 09:12
@jiwon2724 jiwon2724 self-assigned this Nov 17, 2024
Copy link
Contributor

@l5x5l l5x5l left a comment

Choose a reason for hiding this comment

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

확인! 머지 ㄱㄱ 하시죠!

@jiwon2724 jiwon2724 merged commit 551032e into develop Nov 18, 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.

[FEATUR] 공유하기 기능 구현
2 participants