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

The main token with decimal formating is displayed for unrelated asset transfer #461

Closed
Tbaut opened this issue Dec 10, 2023 · 1 comment · Fixed by #462
Closed

The main token with decimal formating is displayed for unrelated asset transfer #461

Tbaut opened this issue Dec 10, 2023 · 1 comment · Fixed by #462
Assignees
@Tbaut
Labels
Priority: 🔴 P0 Added to issues and PRs relating to a critical severity bugs. Type: 🪲 Bug Added to issues and PRs if they are addressing a bug

Comments

@Tbaut
Copy link
Collaborator

Tbaut commented Dec 10, 2023

Here is an example on polkadot asset hub. The extrinsic assets.transferKeepAlive is not about DOTs, although it's a Balance type.

image

we should remove this functionality for the Asset-hubs and Hydra as the same happens, or figure out when this makes sense or not.
@Tbaut Tbaut added Priority: 🔴 P0 Added to issues and PRs relating to a critical severity bugs. Type: 🪲 Bug Added to issues and PRs if they are addressing a bug labels Dec 10, 2023
@Tbaut Tbaut self-assigned this Dec 10, 2023
@Tbaut
Copy link
Collaborator Author

Tbaut commented Dec 10, 2023

pjs/apps has a specific list of overrides for specific extrinsics in https://github.com/polkadot-js/apps/blob/acb87b52e52eda082b3d600abeadfed0f7ca3cc2/packages/react-params/src/overrides.ts#L4

@Tbaut Tbaut mentioned this issue Dec 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Tbaut Tbaut

Labels
Priority: 🔴 P0 Added to issues and PRs relating to a critical severity bugs. Type: 🪲 Bug Added to issues and PRs if they are addressing a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Hotfix: Balance display should not show the token and should not format the amount ChainSafe/Multix
1 participant
@Tbaut