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

Disable NAM in the asset dropdowns + The shield button on NAM in the overview in phases 2/3/4 #1285

Closed
ChrisHoltDesign opened this issue Nov 20, 2024 · 2 comments · Fixed by #1308
Closed

Disable NAM in the asset dropdowns + The shield button on NAM in the overview in phases 2/3/4 #1285

ChrisHoltDesign opened this issue Nov 20, 2024 · 2 comments · Fixed by #1308
Assignees
@euharrison
Labels
App: Namadillo
Milestone
Namadillo Mainnet...

Comments

@ChrisHoltDesign
Copy link

No description provided.

@ChrisHoltDesign
Copy link
Author

Users where trying to move NAM before phase 5 as it was selectable

@euharrison euharrison self-assigned this Nov 21, 2024
@euharrison
Copy link
Collaborator

Currently we are checking for this with asset.display === "nam"

We should have a consistent way to check for NAM asset in the whole Namadillo.

A suggestion is to use the isNamAsset from #1292 and deleting the namadaAsset to avoid inconsistencies across chains

@euharrison euharrison mentioned this issue Nov 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@euharrison euharrison

Labels
App: Namadillo
Projects
None yet
Milestone
Namadillo Mainnet Readiness
Development

Successfully merging a pull request may close this issue.

feat: use namada asset from chain-registry anoma/namada-interface
2 participants
@euharrison @ChrisHoltDesign