[CS2103-T17-3] Automated Insurance Assistant #63
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## master #63 +/- ##
============================================
+ Coverage 72.15% 79.49% +7.33%
- Complexity 399 754 +355
============================================
Files 70 111 +41
Lines 1232 2229 +997
Branches 125 236 +111
============================================
+ Hits 889 1772 +883
- Misses 311 386 +75
- Partials 32 71 +39
Continue to review full report at Codecov.
|
Branch reminder
This issue resolves #130. Now, user guide includes screenshots.
Some explanations are using the wrong date.
Some dates and indexes where wrong in the user guide.
Update user guide with relevent screenshots.
Update AddCommand MESSAGE_USAGE
Fix tag in ToC and command summary
Resolve UserGuide issues from PED
The implementation of the tagging feature should be well-documented in the Developer's guide in order for future developers to make changes to it safety, or it can be referred to when there's some expected bugs. Let's, * Add a detailed class diagram focusing on Tag * Add a brief explanation of the Tag implementation * Add detailed descriptions of all classes related to Tag
The application does not trigger the invalid email response when users do not insert a period between domain name and domain label. Let's do the following to close AY2122S2-2103T-T17#133: - Fix isValidEmail method - Edit tests for isValidEmail
There are certain inconsistencies in the user guide. This may confuse users due to the vague instructions or inaccuracies. Let's do the following: - Fix documentation for the after and within features so that we and close #138 and close #148 - Delete OR example in find features so that we can close #140 - Change defined terms in unlog function to close #142
Sequence Diagrams was not added to the tagging feature implementation. Let's, * Add sequence diagram for removing tags from UniqueTagList * Add sequence diagram for adding tags to UniqueTagList * Add sequence diagram for updating UniqueTagList during setPerson * Update Developer Guide with the new sequence diagram
close #232
Format DG for pdf
close #232
Add page breaks
close #232
Add more page breaks
May page breaks set us free
Update diagram to scope, add border
Arrange glossary alphabetically
<br> was in the wrong position
remove minor issue
Add case-insensitive in ug
Add sections
Add page breaks
Delete one page break
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Automated Insurance Assistant (AIA) is a desktop application for managing contacts. AIA helps insurance agents handle all contact management processes, making sure they always have a snapshot of clients’ profiles on hand, with reminders for events.