-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add 3 new uses cases (add home-buyer, add homes and view home-seller based on priority), update non-functional requirements and glossary #32
Conversation
* 'master' of https://github.com/KhoonSun47/tp: Update EstateEase Developer Guide
* 'master' of https://github.com/KhoonSun47/tp: Update use case UC02: View All
* 'master' of https://github.com/KhoonSun47/tp: Add UC04 to the developer guide
* master: Add UC04 to the developer guide Update use case UC02: View All
- Update DeveloperGuide.md: Add 3 new uses cases (add home-buyer, add homes and view home-seller based on priority) and edit current use case (add home-seller), update non-functional requirements and glossary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Content is okay, just change the numbering in ascending order from top to bottom, starting from UC01 (i.e. UC01, UC02, ... UC##).
docs/DeveloperGuide.md
Outdated
* 3k1. EstateEase shows an error message regarding the entry of incorrect role. <br> | ||
Use case resumes from step 2. | ||
|
||
**Use case: UC00 - Add more homes to home-seller** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😂I think renumber all the UC instead. First one should be UC01 not UC00.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeap updated the numbering of the use case. Thanks for reminding.
docs/DeveloperGuide.md
Outdated
* 3k1. EstateEase shows an error message regarding incorrect format for postal code. <br> | ||
Use case resumes from step 2. | ||
|
||
**Use case: UC00 - Viewing of homes based on home-seller's priority** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change the UC# here too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeap updated the numbering of the use case. Thanks for reminding.
docs/DeveloperGuide.md
Outdated
@@ -385,6 +388,145 @@ Priorities: High (must have) - `* * *`, Medium (nice to have) - `* *`, Low (unli | |||
* 3q1. EstateEase shows an error message regarding the entry of incorrect priority level. <br> | |||
Use case resumes from step 2. | |||
|
|||
**Use case: UC00 - Add home-buyer to user’s contact list** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeap updated the numbering of the use case. Thanks for reminding.
Update DeveloperGuide.md: Update use cases numbering in ascending format
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #32 +/- ##
=========================================
Coverage 75.26% 75.26%
Complexity 419 419
=========================================
Files 71 71
Lines 1338 1338
Branches 126 126
=========================================
Hits 1007 1007
Misses 301 301
Partials 30 30 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Khoon Sun? King Sun.
The developer guide lacks information about new use cases. Hence, adding new use cases ensures that developers have comprehensive documentation about the system's functionality.
The new use cases include adding home-buyer, adding homes and viewing home-seller based on priority.
Non-functional requirements and glossary are also added, so that the developers have a comprehensive understanding of the system's performance, security, usability, and other operational characteristics, in addition to a clear reference for technical terms used throughout the documentation.