Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

[BUG] Incorrect balance & default time captured for transactions. #3457

Closed
1 task done
anikets349 opened this issue Sep 1, 2024 · 5 comments
Closed
1 task done
Labels
approved Approved by the Ivy Wallet team. Ready for dev bug Something isn't working

Comments

@anikets349
Copy link

Please confirm the following

Describe the bug

By default when a transaction is being added, the captured time used to be the current time earlier. Now that does not happen, it is set to some other time that is also disturbing the order of transactions.
Second, about account transfers particularly, it does not reflect the correct balance of the account to which money has been transferred.

To Reproduce

  1. Add a transaction - account transfer
  2. Check if the captured time is current time
  3. Check if the balance of both accounts are updated correctly

Expected behavior

If I add a transaction now, the time by default should be the current time.
Second, the balances of the account(s) must be correctly updated.

Screenshots

No response

App version

v2024.08.25 (195)

Smartphone

OnePlus 12

Additional context

No response

@anikets349 anikets349 added the bug Something isn't working label Sep 1, 2024
@ivywallet
Copy link
Collaborator

Thank you @anikets349 for raising Issue #3457! 🚀
What's next? Read our Contribution Guidelines 📚.

Tagging @ILIYANGERMANOV for review & approval 👀

@ILIYANGERMANOV ILIYANGERMANOV added the approved Approved by the Ivy Wallet team. Ready for dev label Sep 1, 2024
@harsh5424
Copy link

I noticed that the timestamps on all of the transactions changed, and moved back by five and a half hours.
It seems to be a time zone issue, because I'm in India, and that my time zone is +0530, matches perfectly with the time offset on the transaction.

Hope that helps!

@ILIYANGERMANOV
Copy link
Collaborator

@harsh5424 can you try the latest release from GitHub?

@harsh5424
Copy link

All my time stamps seem to be fixed, and adding new transactions gives it the correct time stamp!
That was really quick, thanks guys!

@anikets349
Copy link
Author

Reading @harsh5424's comments, I decided to give things a try again. Works expected.
Thanks for the quick turnaround.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Approved by the Ivy Wallet team. Ready for dev bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants