Skip to content
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

DELETE: Possible ambiguity in allowing assignment via INDEX or NAME #11

Open
whitesnowx opened this issue Apr 5, 2024 · 0 comments
Open

Comments

@whitesnowx
Copy link
Owner

whitesnowx commented Apr 5, 2024

Creates ambiguity in deleting a contact:

  • E.g ambiguity the first or the second contact being deleted
    image.png
steps
clear
add n/1 p/98765432 e/[email protected] a/John street, block 123, #01-01
add n/2 p/98765432 e/[email protected] a/John street, block 123, #01-01
delete 2

Possible solution suggestion:

  • Maybe can restrict the names to not allow pure numerals so 15 will not be a valid name
  • Can limit the usage of assign to only use index
  • Can note that if value passed in is numerical, will be considered as index in the userguide
  • Inform the user that there is a contact with the name that is also a index
@whitesnowx whitesnowx changed the title DELETE: Possible ambiguity in allowing assignment via INDEX or NAME DELETE: Possible ambiguity in allowing assignment via INDEX or NAME Apr 5, 2024
@whitesnowx whitesnowx changed the title DELETE: Possible ambiguity in allowing assignment via INDEX or NAME DELETE: Possible ambiguity in allowing assignment via INDEX or NAME Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant