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

[UX] - Better error handling/messaging to user for transfers not using portal bridge, on portal bridge #246

Closed
mliu-xlabs opened this issue Jul 13, 2023 · 3 comments · Fixed by #253
Labels
enhancement New feature or request need design Need Design'

Comments

@mliu-xlabs
Copy link
Collaborator

mliu-xlabs commented Jul 13, 2023

User Story
As a user, when I am trying to redeem a token for a transfer that did not occur on portal bridge, I want to be shown appropriate error messaging, so I know why I am getting an error message in token bridge

Acceptance Criteria

  • When it is a wallet connect or CCTP or other transfer, show following error message:
    "This is not a token bridge transfer and cannot be recovered here"
@mliu-xlabs
Copy link
Collaborator Author

Need small design here:

Also -- do we have a way to identify if it is a wallet connect or cctp transaction?

@sebastianscatularo
Copy link
Collaborator

I think it will be enough to notify the users that the transaction is not compatible with the portal bridge.

@sebastianscatularo
Copy link
Collaborator

We need the text for the alert message and the location where it should be displayed. As for the rest, we can reuse the current alert style.

@sebastianscatularo sebastianscatularo changed the title Better error handling/messaging to user for transfers not using portal bridge, on portal bridge [UX] - Better error handling/messaging to user for transfers not using portal bridge, on portal bridge Jul 13, 2023
@sebastianscatularo sebastianscatularo added ux enhancement New feature or request labels Jul 13, 2023
@mliu-xlabs mliu-xlabs added need design Need Design' and removed ux labels Jul 18, 2023
@sebastianscatularo sebastianscatularo linked a pull request Jul 18, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request need design Need Design'
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants