search

victorbalssa / abacus

🟠 Abacus: Firefly III mobile application.
https://apps.apple.com/us/app/1627093491
GNU General Public License v3.0
565 stars 42 forks source link

enhancement/required-account-fields #248

Closed br0tda closed 9 months ago

br0tda commented 9 months ago

Added logic to mark as required src/dst account based on transaction type

name: Pull request about: AutocompleteField isRequired attribute title: Logic for required account fields labels: feature assignees: victorbalssa

Is your feature request related to a problem? Please describe. Currently, the source account is always marked as required, even in a 'deposit' transaction

Describe the solution you'd like The source account is marked as required only in 'withdrawal' or 'transfer' transaction. The destination account is marked as required only in 'deposit' or 'transfer' transaction.

Describe alternatives you've considered ...

Additional context ...

br0tda commented 9 months ago

Hi @victorbalssa

Code updated. Thank you!