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

Add hairdresser and client classes #79

Merged
merged 2 commits into from
Sep 30, 2020
Merged

Add hairdresser and client classes #79

merged 2 commits into from
Sep 30, 2020

Conversation

iamjamestan
Copy link

closes #78

@iamjamestan iamjamestan added this to the v1.2 milestone Sep 30, 2020
@iamjamestan iamjamestan self-assigned this Sep 30, 2020
@ya0-yuan
Copy link

LGTM

@ya0-yuan ya0-yuan merged commit 734a0c4 into AY2021S1-CS2103T-T15-1:master Sep 30, 2020
nicktohzyu
nicktohzyu reviewed Oct 7, 2020
View reviewed changes
src/main/java/seedu/address/model/person/Gender.java
import static seedu.address.commons.util.AppUtil.checkArgument;

/**
* Represents a Person's gender in the docX.
Copy link

Choose a reason for hiding this comment

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

docX

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicktohzyu nicktohzyu nicktohzyu left review comments

Assignees

@iamjamestan iamjamestan

Labels
None yet
Projects
None yet
Milestone
v1.2
Development

Successfully merging this pull request may close these issues.

Add hairdresser and client class
3 participants
@iamjamestan @ya0-yuan @nicktohzyu