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

Unintuitive command names #5

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

Unintuitive command names #5

dickongwd opened this issue Nov 17, 2023 · 1 comment

Comments

@dickongwd
Copy link
Owner

dickongwd commented Nov 17, 2023

The app has separate command names for volunteers and events, like vcreate and ecreate, which is fine.

However it starts to get confusing when the command names become elistv, vliste, eremovev (but no vremovee?). Makes it difficult to test for a someone who is completely new to the app.

Moreover, some commands need us to specify the flag eid/, while some doesnt require to.
For example, eedit does not require the index to be in the flag eid/ while the command eaddv requires the index to be in the flag eid/.

@nus-pe-bot
Copy link

nus-pe-bot commented Nov 20, 2023

Team's Response

When we designed the app, we tried to ensure the app usage to be as simplified as possible due to the complexity of our app's features and event-volunteer interactions. For example, when the command name is elistv, it stands for "event list volunteers". Likewise for vliste, it stands for "volunteer list events".

For vremovee, it doesn't make sense due to the logic above so we never implemented it.

The eid/ flag for event-volunteer interaction commands like eaddv part is intentional and necessary in order for us to differentiate between the volunteer and event IDs. For event-only commands like eedit, it only targets one event so there is no need to specify the type of index (eid/).

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: I think "event list volunteers" or "volunteer list events" is not very intuitive/doesn't make a lot of sense, either.

Like how eremovev removes a volunteer from an event to signal that the event has lost a volunteer (from the perspective of the event), vremovee also makes sense because it signals that the volunteer has left the event (from the perspective of the volunteer).

The team says that commands like eedit only targets one event and so there is no need for eid/. However, commands like eaddv also only targets one event. It isn't clear why there is a need for eid/ flag in eaddv. I am trying to suggest that commands like eaddv can follow the format eaddv 1 vid/1 so that all event commands (commands that start with 'e') are consistent in their format.


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