Skip to content

Restore custom URL scheme in mobile deep-link config#449

Open
AnthonyRonning wants to merge 1 commit intomasterfrom
fix/restore-mobile-custom-scheme
Open

Restore custom URL scheme in mobile deep-link config#449
AnthonyRonning wants to merge 1 commit intomasterfrom
fix/restore-mobile-custom-scheme

Conversation

@AnthonyRonning
Copy link
Contributor

@AnthonyRonning AnthonyRonning commented Feb 26, 2026

Adds back the cloud.opensecret.maple custom scheme entry to the mobile deep-link config. This was accidentally removed in 9e6a5a2 (Dec 2, 2025) when fixing Android pathPrefix restrictions.

Without this entry, the Tauri deep-link plugin's build.rs sees no non-app-link mobile domains and strips CFBundleURLTypes from Info.plist, removing the custom URL scheme registration needed for iOS OAuth redirect.

The previous PR (build.rs workaround) is a safety net; this is the proper config-level fix. Does not affect Android -- custom schemes without a host don't generate intent filters.


Open with Devin

Summary by CodeRabbit

Release Notes

  • New Features
    • Added mobile deep-linking support with a custom scheme integration, enabling users to launch the app directly from mobile links.

The cloud.opensecret.maple custom scheme entry was accidentally removed
from the mobile config in 9e6a5a2 when fixing Android pathPrefix. This
tells the deep-link plugin to preserve CFBundleURLTypes in Info.plist
for iOS, complementing the build.rs workaround.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d305190 and 43335d4.

📒 Files selected for processing (1)
  • frontend/src-tauri/tauri.conf.json

📝 Walkthrough

Walkthrough

Adds a new mobile deep-link scheme entry to the Tauri configuration, registering the custom scheme cloud.opensecret.maple with app linking explicitly disabled. This precedes existing HTTPS-based mobile deep-link entries in the configuration file.

Changes

Cohort / File(s) Summary
Mobile Deep-Link Configuration
frontend/src-tauri/tauri.conf.json
Added custom mobile deep-link scheme cloud.opensecret.maple with appLink set to false, enabling custom scheme-based deep-linking for mobile platforms.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A custom scheme hops into place,
cloud.opensecret.maple finds its space,
Mobile deep links now guide the way,
With app links resting at bay,
Configuration blooms today! 🌱

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: restoring a custom URL scheme in the mobile deep-link configuration, which matches the core objective of re-adding the cloud.opensecret.maple scheme entry.

✏️ 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
  • Commit unit tests in branch fix/restore-mobile-custom-scheme

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

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@cloudflare-workers-and-pages
Copy link

Deploying maple with  Cloudflare Pages  Cloudflare Pages

Latest commit: 43335d4
Status:🚫  Build failed.

View logs

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.

1 participant