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

chore: Native logging bridge #278

Draft
wants to merge 1 commit into
base: feature/cdp/project-setup
Choose a base branch
from

Conversation

Ahmed-Ali
Copy link

@Ahmed-Ali Ahmed-Ali commented Jun 23, 2024

This PR introduces the native logging bridge which will be used later to show the native logs in RN dev console

Context

We are rebuilding our React Native (RN) SDK to support Customer.io's Data Pipelines.

Public Changes

  • SDK Changes
    • Modernized our RN package to use the latest configurations and dependencies available in RN version 0.74.2.
    • Simplified our configuration to provide the minimum number of settings that make sense for our customers.
    • Added support to view native debug logs directly in the console alongside other console.* logs.
  • Example Apps
    • Created customer-oriented example apps where you can build, run, debug, and see the SDK usage in the App.tsx file.

Internal Changes

  • SDK Dev Environment Setup
    • Removed the pre-push hook that runs lint, as it can be challenging when lint fails before a push in a PR stack setup. Linting has been moved to the pre-commit hook.
    • Implemented yarn workspace for easier management of example apps.
    • Configured example apps to point to local paths for all native CIO pods, facilitating easier debugging and validation of native changes.
    • Made the native iOS repo a git submodule of this repo, simplifying the process of starting and validating relevant native changes within the RN repo.
    • Unified the example app codebase for both APN and FCM configurations, making it easier to maintain.
    • Included yarn release to ensure a consistent development experience.
    • Added yarn.lock to .gitignore.

QA Setup

  • Dev endpoints for QA can still be set in the example apps with a long-press on the settings button. This allows us to perform QA configurations without affecting customer-oriented examples.

PR Stack:

@Ahmed-Ali Ahmed-Ali force-pushed the feature/cdp/project-setup branch from 16a6af2 to 0a9f81e Compare June 24, 2024 09:32
@Ahmed-Ali Ahmed-Ali force-pushed the feature/cdp/native-logging branch from e7499d6 to 5410957 Compare June 24, 2024 09:32
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.

1 participant