Skip to content

Add react-native-purchases-ui#1540

Merged
Simek merged 2 commits intoreact-native-community:mainfrom
aboedo:patch-1
Mar 10, 2025
Merged

Add react-native-purchases-ui#1540
Simek merged 2 commits intoreact-native-community:mainfrom
aboedo:patch-1

Conversation

@aboedo
Copy link
Copy Markdown
Contributor

@aboedo aboedo commented Mar 10, 2025

📝 Why & how

Adds react-native-purchases-ui package.
RevenueCat ships the react-native-purchases and react-native-purchases-ui packages separately.
The react-native-purchases-ui package hadn't been registered before.

Note: I added this package as a separate entry. There are notes about monorepo handling in the readme but I wasn't entirely sure how to apply them, so let me know if this isn't right.

My interpretation is having a separate listing for each package with the same GitHub URL but different NPM package.

✅ Checklist

  • Added library to react-native-libraries.json
  • Updated library in react-native-libraries.json
  • Documented in this PR how to use the feature or replicate the bug.
  • Documented in this PR how you fixed or created the feature.

Adds react-native-purchases-ui package
@aboedo
Copy link
Copy Markdown
Contributor Author

aboedo commented Mar 10, 2025

If package is a part of monorepo, 'githubUrl' must point to directory where 'package.json' for a given package resides.

the error message is quite clear, I'll fix

…tory that contains the corresponding package.json
Copy link
Copy Markdown
Member

@Simek Simek left a comment

Choose a reason for hiding this comment

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

Hello @aboedo, thanks for adding a missing entry to the directory, LGTM! 👍

@Simek Simek merged commit 24377e3 into react-native-community:main Mar 10, 2025
2 checks passed
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