Skip to content

Conversation

pandablue0809
Copy link
Member

@ihomp
Copy link
Member

ihomp commented Sep 6, 2025

@Anna15170221 can you test this one?

is that what you meant to have?

@Anna15170221
Copy link
Contributor

@ihomp seems working for me

@ihomp
Copy link
Member

ihomp commented Sep 28, 2025

When I open
http://localhost:3000/en/token/rMxCKbEDwqr76QuheSUMdEGf4B9xJ8m5De/524C555344000000000000000000000000000000

I see the the search bar
rMxCKbEDwqr76QuheSUMdEGf4B9xJ8m5De:524C555344000000000000000000000000000000

that is not a userfriendly format to search for currencies. we haven't use that format anywhere in the UI before.

I guess it would be better if we just reuse the same element as in here
http://localhost:3000/en/services/trustline
for selecting a token, with a top up.

No need to overcomplicate the SearchBar.js

Copy link
Member

@ihomp ihomp left a comment

Choose a reason for hiding this comment

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

lets reuse the same element as on services/trustline instead of overcomplicating the searchbar..

it's ok to have the popup..

we shouldn't use the format of issuer:currency

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants