dickongwd / pe

0 stars 0 forks source link

Cannot create event with no roles, but can edit event to create an event with no roles #9

Open dickongwd opened 11 months ago

dickongwd commented 11 months ago

When I enter the command ecreate n/food donation 3 r/ sd/30/11/2023 1200 l/east coastpark dsc/help clean east coast park, I get an error since the r/ parameter is not filled.

However, I can edit an event to have no role requirements by using eedit 1 r/.

A possible suggestion is to make the r/ parameter in ecreate optional.

nus-pe-script commented 11 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Editing one role removes all role

Screenshot 2023-11-17 at 5.08.52 PM.png

Screenshot 2023-11-17 at 5.09.08 PM.png

Similar to Editing one skill removes all skills #4. Refer to it for more details.


[original: nus-cs2103-AY2324S1/pe-interim#4817] [original labels: type.FeatureFlaw severity.Medium]

Their Response to the 'Original' Bug

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

The behaviour is identified and included as a tip in the UG for volunteer skills, but is not very clearly specified for event roles and event materials.

Items for the Tester to Verify

:question: Issue duplicate status

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

Reason for disagreement: What I understand from the other issue is that the user cannot easily add a new role (but must also specify the old roles, if not they will be deleted). It is the same for skills and event materials.

The bug reported here is that events with no roles cannot be created directly, which causes some inconvenience to users who want to create events with no roles. They would have to create an event with a dummy role, and edit that event to clear all roles after that to create an event with no roles.


## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** This is not a documentation bug as it deals with the functionality of the application. The user guide or developer guide is not mentioned here.