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

[CS2103T-F11-1] Hackathon HR Manager #58

Open
wants to merge 219 commits into
base: master
Choose a base branch
from

Conversation

hcwong
Copy link

@hcwong hcwong commented Sep 19, 2019

No description provided.

@hcwong hcwong changed the title [CS2103T-F11-1] Hackathon H&R Manager [CS2103T-F11-1] Hackathon HR Manager Sep 19, 2019
hcwong and others added 5 commits September 23, 2019 17:51
* Update build.gradle file (minor changes)

* Complete set-up for project (as per the Developer Guide)

* Update README.adoc with Travis CI build status

* Update some files to ensure that ./config/travis/run-checks.sh runs fine

* Create the 4 different ListStorages. Create AlfredStorage. Create AlfredStorageManager (halfway done).

* Creation of all necessary Storage classes (with the exception of the TeamListStorage handler classes)

* Added Storage classes for TeamList

* Fix bracket issue

* Update Storage classes with appropriate JSON adaptation

* Update User Guide

* Update README.adoc
* Made changes to fit team's scope.

* Added AddCommands and DeleteCommands.

* Added Edit Commands

* Added List and View commands.

* Corrected repoUrl

* Made changes to follow Joshua's implementations. Contains vaguely-implemented methods.

* Changed parameters from EntityList to Model. Made some changes to follow Joshua's implementations.

* Moved ENTITY_TYPE to a static constant stored in CliSyntax.

* Updated code to use viewMinimal() and viewDetailed().

* Minor changes

* Implemented potential ways of executing each Participant command.

* Merge from upstream

* Updated code to use Model#updateParticipant's return value.

* Removed Commands associated with Issues.

* Removed Commands associated with Issues.

(cherry picked from commit cef6975)

* Imported Entities and other objects.

* Updated AboutUs.adoc and ContactUs.adoc
|=======================================================================

_{More to be added}_

[appendix]
== Use Cases

(For all use cases below, the *System* is the `AddressBook` and the *Actor* is the `user`, unless specified otherwise)
(For all use cases below, the *System* is the `HackathonManager` and the *Actor* is the `user`, unless specified otherwise)

[discrete]

Choose a reason for hiding this comment

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

This should be numbered together with the rest of Use Cases and should not be discrete.


*MSS*

1. User requests a find an Entity of a specific Entity Type.

Choose a reason for hiding this comment

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

Grammar error. Should be "User requests to find ..."

+
Use case ends.

*Extensions*

Choose a reason for hiding this comment

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

Should you add an extension for invalid index? I see in C3 that you can interact with the list through both index and name.

+
Use case ends.

=== Use case: Create an Entity of a specific Entity Type(Participant, Mentor, Team)

Choose a reason for hiding this comment

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

Should contain an extension for when the entity type specified is invalid.

+
Use case resumes from step 4.

=== Use case: Find the top scoring Teams for each category

Choose a reason for hiding this comment

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

This use case shows up twice in the developer guide.


|`* * *` |Admin In-Charge |keep track of winning teams and the prizes won | ensure that the prize-giving ceremony runs smoothly

|`* * *` |Admin In-Charge |make sure that I will be notified on any wrong commands that I type | make sure that I do not accidentally clutter up my list of entries

Choose a reason for hiding this comment

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

If I understood correctly, the notification for wrong commands should be "so that I can re-enter the correct command".


|`* *` |Admin In-Charge |know my sponsor's needs and arrival time | adequately cater to their needs and allocate manpower accordingly

|`* *` |Admin In-Charge |keep track of inventory of swag | make sure they are adequately catered to all participants

Choose a reason for hiding this comment

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

What does swag refer to here? If possible, add it into the glossary for readers to understand. Even though it is in the user guide, it has no definition.

brianyenna and others added 3 commits October 6, 2019 11:41
* Update build.gradle file (minor changes)

* Complete set-up for project (as per the Developer Guide)

* Update README.adoc with Travis CI build status

* Update some files to ensure that ./config/travis/run-checks.sh runs fine

* Create the 4 different ListStorages. Create AlfredStorage. Create AlfredStorageManager (halfway done).

* Creation of all necessary Storage classes (with the exception of the TeamListStorage handler classes)

* Added Storage classes for TeamList

* Fix bracket issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet