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

Invalid Examples for the g/ parameter is not correct #2

Open
Heran9 opened this issue Nov 17, 2023 · 1 comment
Open

Invalid Examples for the g/ parameter is not correct #2

Heran9 opened this issue Nov 17, 2023 · 1 comment

Comments

@Heran9
Copy link
Owner

Heran9 commented Nov 17, 2023

image.png

As you can see, in your description, we cannot put a single number such as 1 as group parameter.

image.png

But I can actually type in add xxx g/1 and it is a correct command.

I'm confusing whether is is a feature flaw or documentaion bug, but from my point of view, you can actually just add a number for the group parameter, so I'll treat it as a documentaion bug.

@nus-pe-bot
Copy link

nus-pe-bot commented Nov 20, 2023

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Confusing parameter information for name of group

image.png

The constraint is that the name of the group can only consist of alphanumeric characters.
Invalid example given: "1"
1 does not violate the constraint because it does not consist any non alphanumeric characters and seems to be a valid input.

If there are more constraints such as the length of group name, it can be included.

Currently, name with the same constraint accept "1" as a valid input.


[original: nus-cs2103-AY2324S1/pe-interim#3602] [original labels: type.DocumentationBug severity.VeryLow]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for pointing this out.

Items for the Tester to Verify

❓ Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

  • I disagree

Reason for disagreement: [replace this with your explanation]


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

2 participants