Marmo / debitum

free and libre IOU tracker for Android
GNU General Public License v3.0
85 stars 7 forks source link

Duplicate People possible #119

Closed pragyanone closed 2 years ago

pragyanone commented 2 years ago

As of now, one can add a Person that already exists.

Also, it would be better if while adding a new Person, a list of matching People appears as one types. Related to #113

Marmo commented 2 years ago

Can you please re-check if both people created really have the same name? Here I get an error message when trying to save a new person with the name of an already existing one.

pragyanone commented 2 years ago

Oh, my bad. Yes, it throws an error.

Marmo commented 2 years ago

As for the second part of your suggestion, I will not add a list of matching people while typing in the add-person-dialog, since the choice/need to create a new person was already identified by the user. So IMO here (in contrast to the add-transaction-dialog in #113) such a list is not needed?/helpful.