Open AndrewOng2066 opened 6 months ago
Thank you for identifying this. However, In defining an invalid index, we would have to go into implementation details (e.g. the index provided is greater than the person list in the app). Implementation details should not be included in the use case. Hence, this is not an issue.
Furthermore, the use case is just to give the reader a good idea of how deleting persons would work in the application. To get a full understanding of "invalid index" we have elaborated on it under the full implementation section of the delete command.
Team chose [response.Rejected
]
Reason for disagreement: [replace this with your explanation]
In the DG: the use case for deleting a person (UC06) stated "The given index is invalid". What does an invalid index mean? It would be better to state the meaning of an invalid index.