Skip to content

[Galaxy]: Add Samsung IAP billing permission to Galaxy manifest#3539

Merged
rickvdl merged 2 commits into
mainfrom
add-galaxy-billing-permission
Jun 5, 2026
Merged

[Galaxy]: Add Samsung IAP billing permission to Galaxy manifest#3539
rickvdl merged 2 commits into
mainfrom
add-galaxy-billing-permission

Conversation

@fire-at-will

@fire-at-will fire-at-will commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds com.samsung.android.iap.permission.BILLING to the Galaxy library manifest so consuming apps inherit it automatically. The Galaxy Store looks for this permission to enable the registration of IAP items, so it's helpful for us to include this permission so that developers don't need to do this manually.

This isn't a breaking change if a developer is already including this permission in their manifest. If they are, the manifest is still able to be merged without issue.


Note

Low Risk
Single manifest permission on a feature module; no runtime logic or security-sensitive code changes.

Overview
Adds the Samsung IAP billing permission (com.samsung.android.iap.permission.BILLING) to the Galaxy library AndroidManifest.xml so apps that depend on that module merge it in automatically.

This avoids requiring each consuming app to declare the same permission manually for Samsung Seller Portal / billing checks.

Reviewed by Cursor Bugbot for commit 486b3dd. Bugbot is set up for automated code reviews on this repo. Configure here.

@fire-at-will
fire-at-will requested a review from a team as a code owner June 4, 2026 18:36
@fire-at-will fire-at-will added the pr:feat A new feature label Jun 4, 2026
@codecov

codecov Bot commented Jun 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.11%. Comparing base (5ddbc02) to head (486b3dd).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3539   +/-   ##
=======================================
  Coverage   80.11%   80.11%           
=======================================
  Files         371      371           
  Lines       15166    15166           
  Branches     2100     2100           
=======================================
  Hits        12150    12150           
  Misses       2166     2166           
  Partials      850      850           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rickvdl rickvdl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice little QoL improvement 🤩

@tonidero tonidero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice! This is indeed a nice improvement!

@rickvdl
rickvdl added this pull request to the merge queue Jun 5, 2026
Merged via the queue into main with commit e1de6c8 Jun 5, 2026
40 checks passed
@rickvdl
rickvdl deleted the add-galaxy-billing-permission branch June 5, 2026 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants