search

byrokrat / giroapp

Command line app for managing autogiro donations.
GNU General Public License v3.0
4 stars 2 forks source link

Read options #113

Closed hanneskod closed 7 years ago

hanneskod commented 7 years ago

Will fix #56

Close #49. See discussion..

hanneskod commented 7 years ago

Adds validation on all input and re-asks questions when validation fails...

One notable change is that AddCommand now defaults payer number to personal id, if no other payer number is specified. The purpose is to make it easier when you manually add a payer and the payer number is the personal id - you then won't have to type i twice, just hit enter..

The drawback is that if you script adding a donor, and you omit the payer number setting then you will no longer get an error, instead it will default to the personal id.

Are you fine with that @nonbinary?

With this addition this pr will close #49 as well as #56.

hanneskod commented 7 years ago

Decision time. Merge. Possible to revert default payer number later if needed...