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

[BUG] calculation mistake #3411

Open
1 task done
Sailesh777 opened this issue Aug 17, 2024 · 9 comments
Open
1 task done

[BUG] calculation mistake #3411

Sailesh777 opened this issue Aug 17, 2024 · 9 comments
Labels
approved Approved by the Ivy Wallet team. Ready for dev bug Something isn't working

Comments

@Sailesh777
Copy link

Please confirm the following

Describe the bug

When I create an report in the expense section through account transfer amount is also calculated even the option is not selected

To Reproduce

Expected behavior

The calculation to be correct, the account transfer amount shouldn't be added up if the option is not selected

Screenshots

Screenshot_2024-08-17-12-06-35-153_com ivy wallet
Screenshot_2024-08-17-12-06-29-112_com ivy wallet
Screenshot_2024-08-17-12-06-17-221_com ivy wallet
Screenshot_2024-08-17-12-06-14-135_com ivy wallet
Screenshot_2024-08-17-12-06-05-661_com ivy wallet
Screenshot_2024-08-17-12-05-57-042_com ivy wallet

App version

V4.6.3 (163)

Smartphone

Redme Note 13 pro plus

Additional context

No response

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

Thank you @Sailesh777 for raising Issue #3411! 🚀
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 Aug 17, 2024
@shamim-emon
Copy link
Member

I'm on it

@ivywallet
Copy link
Collaborator

Thank you for your interest @shamim-emon! 🎉
Issue #3411 is assigned to you. You can work on it! ✅

If you don't want to work on it now, please un-assign yourself so other contributors can take it.

Also, make sure to read our Contribution Guidelines.

@shamim-emon
Copy link
Member

@ILIYANGERMANOV does this bug exist anymore?
Only problem I noticed is if we don't select Transfers as income expense then it's not added to the total even though it's listed( Is it intended behavior or bug).

In-case I'm missing the original bug please let me know how to re-produce.

https://drive.google.com/file/d/160K_jvJCY9q-j0KwNmwZpQjGQnj1_6od/view?usp=sharing

@ILIYANGERMANOV
Copy link
Collaborator

@ILIYANGERMANOV does this bug exist anymore? Only problem I noticed is if we don't select Transfers as income expense then it's not added to the total even though it's listed( Is it intended behavior or bug).

In-case I'm missing the original bug please let me know how to re-produce.

https://drive.google.com/file/d/160K_jvJCY9q-j0KwNmwZpQjGQnj1_6od/view?usp=sharing

By default transfers shouldn't be considered Income/Expense. Transfers are meant to be a zero-value transactions (e.g. moving money from left to right pocket)

@shamim-emon
Copy link
Member

@ILIYANGERMANOV does this bug exist anymore? Only problem I noticed is if we don't select Transfers as income expense then it's not added to the total even though it's listed( Is it intended behavior or bug).

In-case I'm missing the original bug please let me know how to re-produce.

https://drive.google.com/file/d/160K_jvJCY9q-j0KwNmwZpQjGQnj1_6od/view?usp=sharing

By default transfers shouldn't be considered Income/Expense. Transfers are meant to be a zero-value transactions (e.g. moving money from left to right pocket)

Ok,so that is intended behaviour.

How about the bug mentioned in this issue?Is it still there? Or am I missing any required step to re-produce? (video link given above)
.

@ILIYANGERMANOV
Copy link
Collaborator

@shamim-emon I have no clue, the issue author @Sailesh777 should test with the latest APK and tell

@shamim-emon
Copy link
Member

@shamim-emon I have no clue, the issue author @Sailesh777 should test with the latest APK and tell

Ok.
@Sailesh777 please confirm us about it.

@shamim-emon shamim-emon removed their assignment Aug 30, 2024
@Sailesh777
Copy link
Author

Sailesh777 commented Aug 31, 2024 via email

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