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

[CS2113-W10-2] MojoHR #9

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

Conversation

FarmZH98
Copy link

@FarmZH98 FarmZH98 commented Mar 3, 2021

helps an HR personnel keep track of all her emails in the HR department. It allows her to classify her emails and reply emails. It will not give her a notification when a new email comes.
It only allows for one user at a time, multiple users cannot log in at the same time to use the application.

nagiteja referenced this pull request in nagiteja/tp Mar 15, 2021
yanli1215 and others added 28 commits March 30, 2021 17:43
Updated the Login Component of the Developer Guide
Add Utilities component, manual test instructions in DG
Fixed Bugs and modified functions for integration
Comment on lines +75 to +76
The sequence diagram roughly shows how the `compose` operation works.
![ComposeCommand Sequence Diagram](Sequence Diagrams/ComposeCommand.png)

Choose a reason for hiding this comment

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

image

Is the return arrow for "7" necessary since the rest of the methods do not include return arrows?


{Describe the design and implementation of the product. Use UML diagrams and short code snippets where applicable.}
![Architect class diagram](UML diagrams/OverviewClassDiagram.png)
![Architect sequence diagram](UML diagrams/OverviewSequenceDiagram.png)

Choose a reason for hiding this comment

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

image

Is the placement for the Object Deletion notation for the Command Object correct? Would it be clearer if it is placed slightly below the activation bar?

FarmZH98 and others added 30 commits April 12, 2021 21:56
Try to fix windows fail test
# Conflicts:
#	data/12312@gmail.com.json
#	docs/DeveloperGuide.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.