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.

Sign up for GitHub

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

Jump to bottom

Enable deeplinking to transfers #35

Open
jmrossy opened this issue Jul 17, 2023 · 2 comments
Open

Enable deeplinking to transfers #35

jmrossy opened this issue Jul 17, 2023 · 2 comments
Assignees
@jmrossy
Labels
warp-route

Comments

@jmrossy
Copy link
Collaborator

jmrossy commented Jul 17, 2023

#30 and #31 are prerequisites

Current behavior:

No way to access transfer details directly via a link

Desired behavior:

Transfer data is encoded into a URL and deep-links auto-display the transfer details modal. This must work for both the user that initiated the transfer and users that did not.
@jmrossy jmrossy self-assigned this Jul 17, 2023
@jmrossy jmrossy moved this to On-Deck in Hyperlane Tasks Jul 17, 2023
@avious00
Copy link
Contributor

avious00 commented Dec 4, 2023

@jmrossy does this make sense to pull into this sprint?

@jmrossy
Copy link
Collaborator Author

jmrossy commented Dec 5, 2023 •
edited
Loading

@avious00 I don't think so. Personally I don't think this feature is worthwhile but Nam was in favor.
I think it would be confusing for a user to receive a link to a transfer they didn't initiate and then see the details in the tx history modal. If they close the modal, the info would be lost and they'd have to re-open the link.

@avious00 avious00 moved this from Next Sprint to icebox in Hyperlane Tasks Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmrossy jmrossy

Labels
warp-route
Projects
Hyperlane Tasks
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jmrossy @avious00