Skip to content

Update assetlinks.json#1107

Merged
Klakurka merged 1 commit intoPayButton:masterfrom
Fabcien:update_assetlinks
Feb 14, 2026
Merged

Update assetlinks.json#1107
Klakurka merged 1 commit intoPayButton:masterfrom
Fabcien:update_assetlinks

Conversation

@Fabcien
Copy link
Contributor

@Fabcien Fabcien commented Feb 14, 2026

Include the Play Console cert fingerprint that is used to build the production version.

Summary by CodeRabbit

  • Chores
    • Updated Android app verification configuration to support an additional app certificate fingerprint.

Include the Play Console cert fingerprint that is used to build the production version.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 14, 2026

📝 Walkthrough

Walkthrough

The pull request adds a second SHA-256 fingerprint to the Android app configuration in the assetlinks.json file. The existing fingerprint entry is updated with a trailing comma, and a new fingerprint entry is appended to the fingerprints array.

Changes

Cohort / File(s) Summary
Android App Configuration
public/.well-known/assetlinks.json
Added second SHA-256 certificate fingerprint to the android_app fingerprints array, expanding from one to two fingerprint entries.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • feat: assetlinks json #1104: Introduced the assetlinks.json file with a single fingerprint that this PR now extends with a second entry
  • Deeplink file fix #1105: Similar modification to expand the android_app fingerprints array from one to two entries in assetlinks.json

Suggested labels

enhancement (behind the scenes)

Suggested reviewers

  • Klakurka

Poem

🐰 A fingerprint so fine and true,
Now joined by one more in the queue,
The Android links shall surely bind,
Two keys that match, so well-aligned! 🔐✨

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description provides a brief explanation of the change but is missing most required template sections including a related issue reference, test plan, and lacks structural detail. Add the related issue number, document the test plan for verifying the fingerprint functionality, and follow the provided template structure more closely.
Title check ❓ Inconclusive The title 'Update assetlinks.json' is vague and generic, using non-descriptive phrasing that doesn't convey the specific purpose or main change. Consider a more descriptive title like 'Add production Play Console cert fingerprint to assetlinks.json' to clarify the specific change being made.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into master

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Klakurka Klakurka added the enhancement (behind the scenes) Stuff that users won't see label Feb 14, 2026
@Klakurka Klakurka merged commit 0a91e41 into PayButton:master Feb 14, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement (behind the scenes) Stuff that users won't see

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants