Skip to content

feat: [VIDCS-4438] Monorepo consolidation#946

Merged
idelacruz98 merged 3 commits intodevelopfrom
VIDCS-4438
Mar 30, 2026
Merged

feat: [VIDCS-4438] Monorepo consolidation#946
idelacruz98 merged 3 commits intodevelopfrom
VIDCS-4438

Conversation

@joliveraortega
Copy link
Copy Markdown
Contributor

What is this PR doing?

Deprecation note for monorepo consolidation. Merge the two branding-specific RN SDK repositories into a single monorepo to eliminate duplication.

What are the relevant tickets?

VIDCS-4438

Contributing checklist

  • Code must follow existing styling conventions
  • Added a descriptive commit message
  • Sample apps updated if needed

Solves issue(s)

@joliveraortega joliveraortega marked this pull request as ready for review March 17, 2026 15:48
@joliveraortega joliveraortega requested a review from a team as a code owner March 17, 2026 15:48
Copy link
Copy Markdown

@jmontana37 jmontana37 left a comment

Choose a reason for hiding this comment

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

Left a minor comment on order, but message is good. approved.

Comment thread README.md Outdated
@@ -1,3 +1,7 @@
:warning: **This repository has been deprecated in favour of the [Vonage Video React Native SDK](https://github.com/Vonage/vonage-video-react-native-sdk) repository** :warning:

> **Why this change?** We've consolidated our OpenTok and Vonage branded React Native SDKs into a single monorepo to eliminate code duplication. The [Vonage Video React Native SDK](https://github.com/Vonage/vonage-video-react-native-sdk) repository uses brand-aware build configuration to produce both OpenTok and Vonage artifacts from shared sources.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very minor comment, I might switch the 'OpenTok and Vonage' to 'Vonage and OpenTok' just so you lead with the Vonage brand, since that's where it is being merged. But otherwise messaging is fine for me.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@jmontana37 Your comment is addressed now. Thanks!

@idelacruz98 idelacruz98 merged commit d24dd15 into develop Mar 30, 2026
2 checks passed
@idelacruz98 idelacruz98 deleted the VIDCS-4438 branch March 30, 2026 08:39
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