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

Update user guidetable #283

Merged

Conversation

Marcon2509
Copy link

Please review your commands and stuff

@Marcon2509 Marcon2509 added the documentation Improvements or additions to documentation label Nov 4, 2020
@Marcon2509 Marcon2509 added this to the v1.4 milestone Nov 4, 2020
@Marcon2509 Marcon2509 self-assigned this Nov 4, 2020
| | Command Format (Meeting) | Example |
| :--- | :--- | :--- |
| View | add-m q/v b/<BIDDER_ID> p/<PROPERTY_ID> v/<VENUE> t/<DATE> | add-m q/v b/B12 p/P12 v/2 ALBERT PARK t/11-12-2021 |
| Administrative | add-m q/a b/<BIDDER_ID> p/<PROPERTY_ID> v/<VENUE> t/<DATE> | add-m q/a b/B12 p/P12 v/2 ALBERT PARK t/11-12-2021 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would recommend adding the for parameters, for better visuals

| delete-p PROPERTY_ID or delete-p <INDEX_NUMBER> | delete-p P23 or delete-p 5 |
| list-p | list-p |

| Command Format (Bidder) | Example |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bidder and seller can be on top actually

---


### Command Format Summary Tables

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think should push the table summary to the top so its easier for reference at any point in time. (not as instinctive to scroll all the way down, since the users might not know its there at all)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oki will do


💡 DATE indicates the meetingDate of the meeting.


# Features

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can have a pictorial representation of the UI, such as how people take screen shot and demarcate which part of the UI is for what purpose

@Marcon2509 Marcon2509 merged commit 5c24c10 into AY2021S1-CS2103-W14-1:master Nov 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants