-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
There was a problem hiding this 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!!
.../main/java/com/github/se/assocify/ui/screens/treasury/accounting/AccountingDetailedScreen.kt
Show resolved
Hide resolved
...va/com/github/se/assocify/ui/screens/treasury/accounting/balance/BalanceDetailedViewModel.kt
Show resolved
Hide resolved
...rc/main/java/com/github/se/assocify/ui/screens/treasury/receiptstab/receipt/ReceiptScreen.kt
Show resolved
Hide resolved
Quality Gate passedIssues Measures |
There was a problem hiding this 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!!
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.