Skip to content

Conversation

@sorotokin
Copy link
Contributor

This allows different apps and/or different flavors of the same app to work out-of-the box on Android.

Also renamed "landing" to more descriptive "redirect" in the redirect path on Android; on iOS we will need to stage it and do a bit later after tweaking the server config.

Test: manual testing, including iOS.

Fixes #1465

@sorotokin sorotokin linked an issue Jan 2, 2026 that may be closed by this pull request
@sorotokin sorotokin requested a review from davidz25 January 2, 2026 08:59
Copy link
Contributor

@davidz25 davidz25 left a comment

Choose a reason for hiding this comment

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

LGTM with one comment but unit tests will probably keep failing until #1480 has been merged (see that PR for how it also fixes a timebomb-bug).


<!-- 1. HTTPS App Links (default) - Requires .well-known/assetlinks.json -->
<!-- Examples: https://apps.multipaz.org/landing/ -->
<!-- Examples: https://apps.multipaz.org/landing/org.multipaz.testapp/ -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be updated to https://apps.multipaz.org/redirect/org.multipaz.testapp/ also?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@sorotokin sorotokin force-pushed the 1465-redirect-url branch 2 times, most recently from c7c0cc3 to 9f053e4 Compare January 2, 2026 17:19
This allows different apps and/or different flavors of the same app to work out-of-the box
on Android.

Also renamed "landing" to more descriptive "redirect" in the redirect path on Android; on iOS
we will need to stage it and do a bit later after tweaking the server config.

Test: manual testing, including iOS.

Signed-off-by: Peter Sorotokin <sorotokin@gmail.com>
@sorotokin sorotokin merged commit 5719786 into main Jan 2, 2026
3 checks passed
@sorotokin sorotokin deleted the 1465-redirect-url branch January 2, 2026 20:07
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.

Change testapp redirect url to include package name

3 participants