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 #246

Merged
merged 16 commits into from
Nov 11, 2018
Merged

Update Volunteer class #246

merged 16 commits into from
Nov 11, 2018

Conversation

afiqlattif
Copy link

  • Updating of VolunteerId format to NRIC
  • Removal of Birthday restriction

@afiqlattif afiqlattif added the status.Ongoing The issue is currently being worked on. note: remove this label before closing an issue. label Nov 10, 2018
@afiqlattif afiqlattif added this to the v1.4 milestone Nov 10, 2018
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 66.621% when pulling a0dc394 on afiqlattif:update-volunteer-class into 1556348 on CS2103-AY1819S1-W16-2:master.

@afiqlattif afiqlattif added status.Ready The PR is ready to be reviewed. note: remove this label before merging a PR. and removed status.Ongoing The issue is currently being worked on. note: remove this label before closing an issue. labels Nov 11, 2018
@iMarbles iMarbles merged commit 47b9888 into CS2103-AY1819S1-W16-2:master Nov 11, 2018
@afiqlattif afiqlattif deleted the update-volunteer-class branch November 12, 2018 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iMarbles iMarbles iMarbles left review comments

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