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

[Feat/#46] 네이티브 통신 브릿지 구축 #48

Merged
merged 2 commits into from
Feb 6, 2025
Merged

Conversation

sikkzz
Copy link
Collaborator

@sikkzz sikkzz commented Feb 5, 2025

💡 변경사항 & 이슈

모바일 통신 브릿지 구축

✍️ 관련 설명

user가 ios인지 android인지 구분하는 provider랑 해당 환경에 맞게 통신하는 bridge provider 구축했어

일단 노션 참고해서 구성하긴 했는데 맞게 한건지 모르겠고 코드가 더 간소화할 여지가 있는지도 잘 모르겠네

테스트할 방법이 없어서 일단 구성한대로 올려보긴 하는데 혹시 틀린게 있거나 더 좋은 방법이 있으면 얘기해줘!

연결은 home에서 open camera랑 gallery만 우선 연결했는데 이후에 문제 없으면 다 연결하면 될 거 같아 일단은?

⭐️ Review point

  • 통신 방법 맞게 구성 된건지
  • 더 좋은 방법이 있는지
  • 코드 개선점이나 간소화 부분이 있는지

📷 Demo

.

@sikkzz sikkzz added the Feature new feature label Feb 5, 2025
@sikkzz sikkzz self-assigned this Feb 5, 2025
@sikkzz sikkzz requested a review from lgrin-byte as a code owner February 5, 2025 15:02
@sikkzz sikkzz linked an issue Feb 5, 2025 that may be closed by this pull request
1 task
@sikkzz
Copy link
Collaborator Author

sikkzz commented Feb 6, 2025

@lgrin-byte 문제 없을 거 같아? 내가 브릿지 직접 구축하는건 처음이라 테스트할 방법도 없어서 잘 모르겠넹

@lgrin-byte
Copy link
Member

특이사항은 딱히 없어보이는데 앱에서 웹띄우고 확인해봐야할 것 같당 어차피 웹배포는 금방하니까!

@sikkzz sikkzz merged commit c96e7c9 into develop Feb 6, 2025
1 check passed
@sikkzz sikkzz deleted the feat/#46 branch February 6, 2025 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 네이티브 통신 브릿지 구축
2 participants