Skip to content

Comments

Restrict deep link intent filter to supported paths#1075

Open
gregmarra wants to merge 1 commit intomainfrom
restrict-deep-links
Open

Restrict deep link intent filter to supported paths#1075
gregmarra wants to merge 1 commit intomainfrom
restrict-deep-links

Conversation

@gregmarra
Copy link
Contributor

Summary

  • Scoped the AndroidManifest intent filter from catching all thebluealliance.com URLs to only paths the app can handle: /event/, /events, /match/, /team/, /teams
  • Added deep link support for /events and /teams list screens (with {year} and {page} URI patterns for future use)
  • Unsupported URLs (/admin, /gameday, /account, etc.) now correctly open in the browser instead of the app

Test plan

  • adb shell am start ... /event/2026test → Event detail screen
  • adb shell am start ... /team/177 → Team detail screen
  • adb shell am start ... /events → Events tab
  • adb shell am start ... /teams → Teams tab
  • adb shell am start ... /admin → "unable to resolve Intent" (not caught)
  • adb shell am start ... /gameday → "unable to resolve Intent" (not caught)
  • ./gradlew :app:assembleDebug builds clean

🤖 Generated with Claude Code

The manifest previously caught all https://www.thebluealliance.com URLs,
but the app only handles /event/, /match/, and /team/ routes. Unsupported
URLs like /admin or /gameday would open the app to a broken state instead
of the browser.

Scope the intent filter to only intercept paths the app can handle, and
add deep link support for /events and /teams list screens.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor

@bherbst bherbst left a comment

Choose a reason for hiding this comment

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

Ha, this was on my list too!

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