Skip to content

Conversation

@neocybereth
Copy link
Collaborator

No description provided.

@neocybereth neocybereth force-pushed the feat/new-transfer-module-v2 branch 2 times, most recently from 61e06e9 to 0261268 Compare June 9, 2025 23:51
@neocybereth neocybereth force-pushed the feat/new-transfer-module-v2 branch from 4fc7426 to b787e15 Compare July 10, 2025 05:43
@neocybereth neocybereth force-pushed the feat/new-transfer-module-v2 branch 2 times, most recently from 5be15ab to e3b9098 Compare July 24, 2025 04:47
@neocybereth neocybereth force-pushed the feat/new-transfer-module-v2 branch 4 times, most recently from ed4a2b8 to a0adbe3 Compare August 22, 2025 08:19
@brentstone
Copy link

image

Couple things to fix in this image for the Shield tab:

  • The tokens shown are all in my transparent Namada currently. So should they have the smaller bottom-right image indicating the network they are from? At minimum, the NAM token should not have the Osmosis image.
  • In the left bar, if I select the Namada network, then none of these tokens show up, even though they are all on Namada.
  • Also note that my Keplr is not even connected rn
  • The tokens only show up under All networks

@brentstone
Copy link

It looks like only All Networks in the From A to Z tab is the only one that works regardless of which address is selected (I've tried Namada transparent, shielded, and Keplr)

@neocybereth neocybereth force-pushed the feat/new-transfer-module-v2 branch 2 times, most recently from acc1726 to 9974774 Compare October 14, 2025 00:33
@neocybereth neocybereth force-pushed the feat/new-transfer-module-v2 branch 2 times, most recently from 70495d4 to 6935a20 Compare October 27, 2025 21:42
@brentstone brentstone added this to the Current Priorities milestone Oct 27, 2025
}
newParams.set(
params.asset,
selectedAssetWithAmount.asset.symbol || ""
Copy link
Collaborator

Choose a reason for hiding this comment

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

No need to fallback to ""

Copy link
Collaborator

Choose a reason for hiding this comment

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

Still || ""

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@mateuszjasiuk mateuszjasiuk force-pushed the feat/new-transfer-module-v2 branch from 749776b to 0ee1b38 Compare November 5, 2025 12:50
@neocybereth neocybereth merged commit ca440de into main Nov 5, 2025
6 checks passed
@neocybereth neocybereth deleted the feat/new-transfer-module-v2 branch November 5, 2025 19:21
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.

5 participants