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.

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

Link balance items to receipt #373

Merged
merged 16 commits into from
May 29, 2024
Merged

Link balance items to receipt #373

merged 16 commits into from
May 29, 2024

Conversation

sarahbadr17
Copy link
Contributor

@sarahbadr17 sarahbadr17 commented May 24, 2024

The balance items are now correlated with receipts.

When you change the status of a balance items, the status icon is directly changed in the BalanceDetailedScreen and in the ReceiptScreen.

You can no longer change the status of a receipt in the ReceiptScreen. Only the treasurer can do that and it makes sense do that only in Balance.

You can choose to link a receipt to the balance item or not. When you chose not to, you can choose the "No receipt" value and input your own amount. However, if you choose a receipt, the amount will be linked to the amount of the receipt and the status of the balance item will be linked to the status of the receipt.

@sarahbadr17 sarahbadr17 self-assigned this May 24, 2024
@ortolino ortolino self-requested a review May 27, 2024 19:52
@sarahbadr17 sarahbadr17 marked this pull request as ready for review May 29, 2024 10:21
Copy link
Contributor

@ortolino ortolino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good PR overall, I have a few questions that depend on it but however I am happy with what has been done. Good job!!

Copy link

sonarcloud bot commented May 29, 2024

Copy link
Contributor

@ortolino ortolino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All my doubts are solved, thank you for your explanations!!

@sarahbadr17 sarahbadr17 merged commit 158542e into main May 29, 2024
3 checks passed
@Mai-LinhC Mai-LinhC linked an issue Jun 1, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Link receipt to balance items
2 participants