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

Names, URL and info of apps connected to the wallet are not consistent across the wallet #1149

Closed
HoTandy opened this issue Mar 19, 2024 · 0 comments · Fixed by #1197
Closed
Assignees

Comments

@HoTandy
Copy link
Contributor

HoTandy commented Mar 19, 2024

What version of Wallet are you using?

0.15.2

What is happening?

When an app is connected to the wallet, it's not so consistent across screens and not clear what app is connected.

I the first screenshot below under Connected Apps, we see only the URL of the app that's connected along with some info text that states 2 accounts connected

image

If I edit the app above, I go into the screen shown below, here I see 2 lines on the connected app, the first line stating what the app is, the Fuel Bridge and the second line with the URL of the app. That inconsistent with what we see in the first image

image

steps to reproduce

Connect the bridge to the wallet

How it should behave?

The app shown in the 2 images above:

  1. have them consistent in what is displayed.
  2. Show the name of the app rather than the URL so the user knows what app is connected
  3. Maybe show all 3 pieces of information across 3 lines

What os are you seeing the problem on?

MacOS

What browser?

Chrome

What is your browser version?

Version 122.0.6261.112 (Official Build) (x86_64)

@HoTandy HoTandy added bug Something isn't working triage labels Mar 19, 2024
@LuizAsFight LuizAsFight added feat and removed bug Something isn't working labels Mar 25, 2024
@helciofranco helciofranco self-assigned this Mar 25, 2024
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 a pull request may close this issue.

3 participants