Currently, if a user adds a service + username that exists already... the behavior is unclear.
I think we should show an error message, like "Service and username already existing".
In principle we could replace the existing one, but that would create mess because if the group changes, the secret also changes. So I think that guarding against add is the easiest solution.
Currently, if a user adds a service + username that exists already... the behavior is unclear.
I think we should show an error message, like "Service and username already existing".
In principle we could replace the existing one, but that would create mess because if the group changes, the secret also changes. So I think that guarding against add is the easiest solution.
We should also add tests.