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.

Sign up for GitHub

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

Jump to bottom

[PE-D][Tester B] Edit command does not allow edit to be done on contact with the same name #198

Closed
nus-se-bot opened this issue Oct 29, 2022 · 1 comment · Fixed by #219
Closed

[PE-D][Tester B] Edit command does not allow edit to be done on contact with the same name #198

nus-se-bot opened this issue Oct 29, 2022 · 1 comment · Fixed by #219
Assignees
@florentianayuwono
Labels
bug Something isn't working priority.High High prioritization
Milestone
v1.4

Comments

@nus-se-bot
Copy link

image.png

In my opinion, I think edit should be allowed unless all the other fields are the same. Quite common to have the same name

Labels: type.FeatureFlaw severity.Medium
original: PeiYee88/ped#11
@Ugholaf
Copy link

Ugholaf commented Oct 30, 2022

Similar issue #210

@Ugholaf Ugholaf added bug Something isn't working priority.High High prioritization labels Oct 30, 2022
@ChongCheeKaiClarence ChongCheeKaiClarence added this to the v1.4 milestone Nov 1, 2022
@florentianayuwono florentianayuwono linked a pull request Nov 1, 2022 that will close this issue
Search Feature and Documentation related bug fix #219
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@florentianayuwono florentianayuwono

Labels
bug Something isn't working priority.High High prioritization
Projects
None yet
Milestone
v1.4
Development

Successfully merging a pull request may close this issue.

Search Feature and Documentation related bug fix florentianayuwono/tp
4 participants
@nus-se-bot @Ugholaf @florentianayuwono @ChongCheeKaiClarence