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

[BUG] Incorrect Time Display and Account Balance Issue(critical issue) #3434

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

Comments

@asadmash
Copy link

Please confirm the following

Describe the bug

When adding a new entry, the entry time initially shows the current time, but after clicking 'Add,' it displays a time that is 6 hours earlier. Additionally, if the timing is changed manually, the entry reflects the correct time. However, the income, expenses, or transfer amounts do not update the account balance accordingly.

To Reproduce

  1. Add a new entry with the default time (current time).
  2. Click 'Add' to save the entry.
  3. Observe that the entry time now shows a time that is 6 hours earlier.
  4. Manually adjust the time of the entry.
  5. Observe that the entry now shows the correct time.
  6. Check the account balance to see if the income, expenses, or transfer amounts have been updated.

Expected behavior

The entry time should remain consistent with the current time unless manually changed.

The account balance should be correctly updated based on the income, expenses, or transfer amounts entered.

Screenshots

No response

App version

v2024.08.25(195) from play store & latest demo apk from TG.

Smartphone

symphony z40, android 10

Additional context

No response

@asadmash asadmash added the bug Something isn't working label Aug 27, 2024
@ivywallet
Copy link
Collaborator

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

Tagging @ILIYANGERMANOV for review & approval 👀

@rodrigomatosc
Copy link
Contributor

I'm on it

@ivywallet
Copy link
Collaborator

⚠️ Hey @rodrigomatosc, this issue is not approved, yet.
@ILIYANGERMANOV must approve it first.

Also, make sure to read our Contribution Guidelines.

@ILIYANGERMANOV ILIYANGERMANOV added the approved Approved by the Ivy Wallet team. Ready for dev label Aug 28, 2024
@ILIYANGERMANOV
Copy link
Collaborator

Appreciate. Hey @rodrigomatosc @shamim-emon I'm also working on the time problems in general. I'm trying to apply an app-wide fix. However, feel free to give these a shot - just heads up that we might have merge conflicts

@shamim-emon
Copy link
Member

Appreciate. Hey @rodrigomatosc @shamim-emon I'm also working on the time problems in general. I'm trying to apply an app-wide fix. However, feel free to give these a shot - just heads up that we might have merge conflicts

Alright!

@shamim-emon
Copy link
Member

Appreciate. Hey @rodrigomatosc @shamim-emon I'm also working on the time problems in general. I'm trying to apply an app-wide fix. However, feel free to give these a shot - just heads up that we might have merge conflicts

It'd be even better if we could split modules among ourselves?
What do you say @ILIYANGERMANOV?

@ILIYANGERMANOV
Copy link
Collaborator

Appreciate. Hey @rodrigomatosc @shamim-emon I'm also working on the time problems in general. I'm trying to apply an app-wide fix. However, feel free to give these a shot - just heads up that we might have merge conflicts

It'd be even better if we could split modules among ourselves? What do you say @ILIYANGERMANOV?

It's hard to split because you change one place to Instant the entire app wont' build :/ The change will propagate among all usages. I think to work a couple of hours today will make some buildable progress. if you want you can start building on top of it. Here's my PR:
#3435

I created the APIs that we should use across the app.

@asadmash
Copy link
Author

Problem solved. Thanks to the developers and contributors.🤍

@5VPH3R
Copy link

5VPH3R commented Aug 30, 2024

When will the fix be merged so we can receive the update via Google Play Store?

@5VPH3R
Copy link

5VPH3R commented Sep 1, 2024

Problem solved. Thanks to the developers and contributors.🤍

Hello @asadmash
The fix still didn't reach us on Google Play Store
I'm still facing issues on GMT +3
Any ETA for getting the fix?

@ILIYANGERMANOV
Copy link
Collaborator

ILIYANGERMANOV commented Sep 1, 2024

Problem solved. Thanks to the developers and contributors.🤍

Hello @asadmash The fix still didn't reach us on Google Play Store I'm still facing issues on GMT +3 Any ETA for getting the fix?

It's on Google Play Beta, I'll promote it to prod and it'll be live when Google approves it

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

7 participants
@ILIYANGERMANOV @rodrigomatosc @shamim-emon @5VPH3R @asadmash @ivywallet and others