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

Ability to disable amazon's DRM #2849

Merged

Conversation

KonstantinosKoulaxizis
Copy link
Contributor

@KonstantinosKoulaxizis KonstantinosKoulaxizis commented Sep 19, 2024

In some cases, users want to manage the DRM with their own solution and not use Amazon's DRM. In the past, this was possible by unchecking the DRM option from Amazon's Developer console, but now you can only do it from their SDK.

The react-native-iap module by default will enable Amazon's DRM, but if a user doesn't want to leverage it, there is no option to disable it.

With this PR a user can easily disable it by adding isAmazonDrmEnabled=false to their gradle.properties file.
The default will be true

Screenshot 2024-09-19 at 13 32 53

@@ -19,7 +19,8 @@ android {
targetSdkVersion rootProject.ext.targetSdkVersion
versionCode 34
versionName "1.16.2"
buildConfigField "boolean", "IS_NEW_ARCHITECTURE_ENABLED", isNewArchitectureEnabled().toString()`;
buildConfigField "boolean", "IS_NEW_ARCHITECTURE_ENABLED", isNewArchitectureEnabled().toString()
buildConfigField "boolean", "ENABLE_AMAZON_DRM", getExtOrDefault("enableAmazonDrm")`;
Copy link
Owner

Choose a reason for hiding this comment

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

Could we make this look more like a boolean? Something like isAmazonDrmEnabled would be great.

@hyochan
Copy link
Owner

hyochan commented Sep 21, 2024

Additionally, providing documentation related to this feature would be great

@KonstantinosKoulaxizis KonstantinosKoulaxizis added the 📖 documentation Improvements or additions to documentation label Sep 23, 2024
Copy link
Owner

@hyochan hyochan left a comment

Choose a reason for hiding this comment

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

LGTM!!

@hyochan hyochan merged commit b7aa8fa into hyochan:main Oct 17, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ⓐ amazon 🤖 android Related to android 📖 documentation Improvements or additions to documentation 🎯 feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants