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

Update Volunteer Class tests #254

Merged
merged 4 commits into from
Nov 12, 2018
Merged

Update Volunteer Class tests #254

merged 4 commits into from
Nov 12, 2018

Conversation

afiqlattif
Copy link

  • Added new Volunteer Class tests
  • Updated Birthday to check that Birthday must be before current day
  • Fixed formatting of Add Command message

@afiqlattif afiqlattif added this to the v1.4 milestone Nov 12, 2018
@afiqlattif afiqlattif added the status.Ready The PR is ready to be reviewed. note: remove this label before merging a PR. label Nov 12, 2018
@coveralls
Copy link

coveralls commented Nov 12, 2018 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 67.598% when pulling a36461f on afiqlattif:update-volunteer-tests into 13166cd on CS2103-AY1819S1-W16-2:master.

@iMarbles iMarbles merged commit 4886b00 into CS2103-AY1819S1-W16-2:master Nov 12, 2018
@afiqlattif afiqlattif deleted the update-volunteer-tests branch November 14, 2018 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iMarbles iMarbles Awaiting requested review from iMarbles

@Kratious Kratious Awaiting requested review from Kratious

Assignees
No one assigned
Labels
status.Ready The PR is ready to be reviewed. note: remove this label before merging a PR.
Projects
None yet
Milestone
v1.4
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@afiqlattif @coveralls @iMarbles