Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Wallet part of the https deeplinking #82

Merged
merged 7 commits into from
Nov 30, 2023

Conversation

Tommylans
Copy link
Member

@Tommylans Tommylans commented Nov 22, 2023

I couldn't tested yet on Android for some reason I can't get it to start.

I also added the workaround for the didcomm://https://agent.example.com/didcomm?oob=xxxxx so that is also possible now.

The workaround is something what I found by accident. I tried more things but all of them were not possible. Some were not possible because the routing was earlier than the deeplink detection.

Also didn't find anything about it in React router, solite and tamagui.

Tommylans and others added 2 commits November 21, 2023 12:34
Signed-off-by: Tom Lanser <[email protected]>
Signed-off-by: Tom Lanser <[email protected]>
@Tommylans Tommylans marked this pull request as ready for review November 30, 2023 15:13
@Tommylans Tommylans enabled auto-merge (squash) November 30, 2023 15:13
@Tommylans Tommylans disabled auto-merge November 30, 2023 15:13
@Tommylans Tommylans requested a review from janrtvld November 30, 2023 15:26
Copy link
Contributor

@janrtvld janrtvld left a comment

Choose a reason for hiding this comment

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

Maybe clarify a bit more in a comment in _layout.tsx what exactly this does (not a lot just some more info besides that this is a workaround for deeplinking).

apps/expo/app/_layout.tsx Outdated Show resolved Hide resolved
packages/agent/src/parsers.ts Show resolved Hide resolved
@Tommylans Tommylans merged commit 7ddd88f into main Nov 30, 2023
1 check passed
@Tommylans Tommylans deleted the feature/https-deeplinking branch November 30, 2023 15:44
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