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: ANLSPI-22113 auto web js sdk bridge #335

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

YoloMao
Copy link
Collaborator

@YoloMao YoloMao commented Oct 12, 2024

PR 内容

运行时,通过以下接口开启 Web JS SDK 自动注入 (Demo 已经改好了):

GrowingHybridModule.sharedInstance.autoJsSdkInject = YES; // default NO

GrowingHybridModule.sharedInstance.autoBridgeEnabled (默认为 YES) 等原接口互不干扰,仅当 2 个配置都为 YES 或者内嵌页已经集成了 Web JS SDK 时,Hybrid 打通

测试步骤

  • 测试 Cocoapods 集成时,Podfile 配置为 use_frameworks!use_modular_headers! 等等情况下,正确加载 Apple SDK 内置的 Web JS SDK
  • 测试 SwiftPM 集成时,正确加载 SDK 内置的 Web JS SDK,附:SPM 集成说明
  • 测试 Web JS SDK 正常运行且打通

注意

  • 内置的 Web JS SDK 每次更新需要 Apple SDK 发版
  • 仅 iOS 平台下可开启 Web JS SDK 自动注入

Copy link

codecov bot commented Oct 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.14%. Comparing base (9b668ef) to head (3a09a97).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #335      +/-   ##
==========================================
+ Coverage   82.21%   83.14%   +0.92%     
==========================================
  Files         136      136              
  Lines       10182    10227      +45     
==========================================
+ Hits         8371     8503     +132     
+ Misses       1811     1724      -87     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@YoloMao YoloMao force-pushed the feat/auto_web_sdk branch 6 times, most recently from 569bbad to 02f2713 Compare October 16, 2024 07:35
Copy link

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.

3 participants