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

[Bugfix] Some accounts don't show their Trust Lines #162

Closed
WietseWind opened this issue May 8, 2020 · 2 comments
Closed

[Bugfix] Some accounts don't show their Trust Lines #162

WietseWind opened this issue May 8, 2020 · 2 comments
Assignees
@N3TC4T
Labels
App iOS/Android App Reported bug Reported bug (existing or introduced at update)
Projects
XUMM 0.X, 1.X & 2.1

Comments

@WietseWind
Copy link
Member

Some accounts don't show Trust Lines, not in home screen and not in Explain balance tab.

Sample (eg. add Read Only):
rDV97E5whZbKvcfgmnqA7whvKx1ioqg6nT
While it does have trust lines:
https://bithomp.com/explorer/rDV97E5whZbKvcfgmnqA7whvKx1ioqg6nT

Apparently this is because of the scientific notation of the Limit in case of a very high limit on the Trust Line.
@WietseWind WietseWind added the App iOS/Android App label May 8, 2020
@WietseWind WietseWind added the Reported bug Reported bug (existing or introduced at update) label May 8, 2020
@WietseWind WietseWind added this to Done, Test (for 0.4.2 Beta release) in XUMM 0.X, 1.X & 2.1 May 8, 2020
@WietseWind
Copy link
Member Author

WietseWind commented May 8, 2020 •
edited

Duplicate: #159

TrustLines with hight limit will cause crash in the events list 

[Error: invalid value '1000000000000000e-30', should be a number matching (^-?[0-9]*.?[0-9]*$).]

Suggestion:

^[+-]?\d+(?:[\.]*\d*(?:[eE][+-]?\d+)?)?$

^^ parses number & scientific notation

@WietseWind WietseWind mentioned this issue May 8, 2020
Closed
2 tasks
@WietseWind WietseWind moved this from Done, Test (for 0.4.2 Beta release) to 0.4.2 (Late 0.4 post-release) in XUMM 0.X, 1.X & 2.1 May 8, 2020
@WietseWind
Copy link
Member Author

https://app.intercom.com/a/apps/nq9rb5du/inbox/inbox/all/conversations/26990075470

@N3TC4T N3TC4T moved this from Current sprint (0.5.0) to Done, Test (0.5.0) in XUMM 0.X, 1.X & 2.1 May 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@N3TC4T N3TC4T

Labels
App iOS/Android App Reported bug Reported bug (existing or introduced at update)
Projects
XUMM 0.X, 1.X & 2.1
  
Released (0.4.9/0.5.0)
Milestone
No milestone
Development

No branches or pull requests
2 participants
@WietseWind @N3TC4T