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.

Sign up for GitHub

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

Jump to bottom

Updates to developer guide. #120

Merged
merged 1 commit into from Oct 30, 2018
Merged

Updates to developer guide. #120

merged 1 commit into from Oct 30, 2018

Conversation

kianhong95
Copy link
Collaborator

Updated developer guide with user stories and use case for clock in and clock out commands

@kianhong95 kianhong95 added priority.high Must do severity.low A flaw that is unlikely to affect normal operations of the product. status.review-pending The pull request is ready to be reviewed. labels Oct 30, 2018
@kianhong95 kianhong95 added this to the v1.3 milestone Oct 30, 2018
@kianhong95 kianhong95 self-assigned this Oct 30, 2018
AngWM
AngWM approved these changes Oct 30, 2018
View reviewed changes
Copy link
Collaborator

@AngWM AngWM left a comment

Choose a reason for hiding this comment

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

Looks ready to be merged

@px1099 px1099 added status.completed The pull request has been reviewed. and removed status.review-pending The pull request is ready to be reviewed. labels Oct 30, 2018
px1099
px1099 approved these changes Oct 30, 2018
View reviewed changes
Copy link
Collaborator

@px1099 px1099 left a comment

Choose a reason for hiding this comment

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

Great update to the employee use case

@px1099 px1099 merged commit 798b132 into CS2113-AY1819S1-F09-2:master Oct 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AngWM AngWM AngWM approved these changes

@px1099 px1099 px1099 approved these changes

@kangmingtay kangmingtay Awaiting requested review from kangmingtay

@SalsabilTasnia SalsabilTasnia Awaiting requested review from SalsabilTasnia

Assignees

@kianhong95 kianhong95

Labels
priority.high Must do severity.low A flaw that is unlikely to affect normal operations of the product. status.completed The pull request has been reviewed.
Projects
None yet
Milestone
v1.3
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kianhong95 @AngWM @px1099