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

Fix incorrect pending transaction value #399

Closed
alexfreska opened this issue May 24, 2024 · 3 comments · Fixed by #415
Closed

Fix incorrect pending transaction value #399

alexfreska opened this issue May 24, 2024 · 3 comments · Fixed by #415
Assignees
@alexfreska
Labels
ui User interface related issues
Milestone
v1.1.1

Comments

@alexfreska
Copy link
Member

@alexfreska alexfreska added this to Sia May 24, 2024
@alexfreska alexfreska self-assigned this May 24, 2024
@alexfreska alexfreska converted this from a draft issue May 24, 2024
@alexfreska alexfreska moved this from Triage to Backlog in Sia May 24, 2024
@serveriukai
Copy link

adding screenshot here:
last three transactions were all -1000 SC
strange-transfers

@n8maninger n8maninger added the ui User interface related issues label Jun 3, 2024
@ChrisSchinnerl ChrisSchinnerl moved this from Backlog to Todo in Sia Jun 5, 2024
@n8maninger n8maninger added this to the v1.1.1 milestone Jun 10, 2024
@alexfreska
Copy link
Member Author

@n8maninger the siacoin values are calculated as BigNumber(t.inflow).minus(t.outflow) for both pending and regular transactions - is this approach incorrect? I just tested sending 50 SC, and it looks like inflow is 49 and outflow is 0 in the pending transaction data.
Screenshot 2024-06-12 at 4 10 45 PM

@alexfreska alexfreska moved this from Todo to In Progress in Sia Jun 12, 2024
@alexfreska
Copy link
Member Author

Ah interesting, @n8maninger if I do the opposite and send 100 siacoin to hostd, the incoming 100 siacoin value shows as inflow 100, outflow 0.

@n8maninger n8maninger mentioned this issue Jun 12, 2024
Merged
@n8maninger n8maninger moved this from In Progress to In Review in Sia Jun 12, 2024
@github-project-automation github-project-automation bot moved this from In Review to Done in Sia Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexfreska alexfreska

Labels
ui User interface related issues
Projects
Sia
Status: Done
Milestone
v1.1.1
Development

Successfully merging a pull request may close this issue.

Fix tpool unconfirmed SiaFoundation/hostd
3 participants
@alexfreska @n8maninger @serveriukai