Skip to content

Comments

[feature] 동아리 상세 웹뷰 네이티브 공유하기 핸들러 구현#4

Merged
SeongHoonC merged 1 commit intomainfrom
feature/share-link-bridge
Jan 24, 2026
Merged

[feature] 동아리 상세 웹뷰 네이티브 공유하기 핸들러 구현#4
SeongHoonC merged 1 commit intomainfrom
feature/share-link-bridge

Conversation

@seongwon030
Copy link
Member

@seongwon030 seongwon030 commented Jan 22, 2026

작업

  • 웹뷰에서 전달받은 'SHARE' 메시지를 처리하여 네이티브 공유하기(Share.share) 기능을 실행하는 로직 추가
  • WebView 컴포넌트에 onMessage 핸들러 연결

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 클럽 상세 페이지에서 공유 기능 추가됨
  • 스타일

    • UI 요소의 일관성 개선 및 색상 표준화

✏️ Tip: You can customize this high-level summary in your review settings.

- 웹뷰에서 전달받은 'SHARE' 메시지를 처리하여 네이티브 공유하기(Share.share) 기능을 실행하는 로직 추가
- WebView 컴포넌트에 onMessage 핸들러 연결
@coderabbitai
Copy link

coderabbitai bot commented Jan 22, 2026

걸어보기

클럽 상세 화면 컴포넌트에 WebView 메시지 지원을 추가했습니다. SHARE 메시지를 처리하는 핸들러를 구현하고 React Native Share API와 통합하여 제목, 텍스트, URL을 공유할 수 있도록 했습니다. 문자열 인용 부호 일관성 개선 및 스타일 미세 조정도 포함되었습니다.

변경 사항

응집 / 파일(들) 변경 요약
WebView 메시징 및 공유 기능
ui/club-detail/club-detail-screen.tsx
WebViewMessageEvent 타입 임포트 추가, onMessage 콜백 연결, SHARE 메시지 처리를 위한 handleMessage 함수 구현, React Native Share API를 통한 공유 기능 통합
코드 스타일 및 형식화
ui/club-detail/club-detail-screen.tsx
문자열 리터럴 인용 부호를 작은따옴표에서 큰따옴표로 통일, 공백 및 후행 쉼표 조정, 헤더 테두리 색상 16진수를 소문자로 변경(#f0f0f0)

시퀀스 다이어그램

sequenceDiagram
    participant WebView
    participant ClubDetailScreen as Club Detail Screen
    participant ShareAPI as React Native Share

    WebView->>ClubDetailScreen: onMessage(SHARE event)
    activate ClubDetailScreen
    ClubDetailScreen->>ClubDetailScreen: handleMessage(event)
    ClubDetailScreen->>ClubDetailScreen: Extract title, text, URL
    deactivate ClubDetailScreen
    
    ClubDetailScreen->>ShareAPI: Share.share({title, text, url})
    activate ShareAPI
    ShareAPI->>ShareAPI: Show share sheet
    ShareAPI-->>ClubDetailScreen: Share result
    deactivate ShareAPI
Loading

예상 코드 리뷰 노력

🎯 3 (보통) | ⏱️ ~20 분

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경사항의 주요 기능을 명확하게 설명하고 있습니다. WebView의 SHARE 메시지 처리 및 네이티브 공유하기 핸들러 구현이 주요 변경사항이며, 제목도 이를 정확히 반영하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Collaborator

@SeongHoonC SeongHoonC left a comment

Choose a reason for hiding this comment

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

필요없는 부분이 추가된 것 같은데 변경해도 문제 없을 것 같아 머지하고 넘어가겠습니다. 다음부터 확인해주세요

@SeongHoonC SeongHoonC merged commit f602008 into main Jan 24, 2026
1 check passed
@seongwon030
Copy link
Member Author

필요없는 부분이 추가된 것 같은데 변경해도 문제 없을 것 같아 머지하고 넘어가겠습니다. 다음부터 확인해주세요

다음부터 확인하고 올리겠습니다!

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.

2 participants