Ong Han Wei's Project Portfolio Page
Project: MyInsuRec
MyInsuRec is a desktop app for financial advisors to manage and organize their clients and meetings, helping financial advisors stay connected with their clients.
Given below are my contributions to the project.
-
Code contributed: RepoSense link
- Enhancements Implemented:
- Refactored most
personandaddressbooktoclientand MyInsuRec` respectively. #116 This task alone was fairly simple to complete after looking at tutorials from the textbooks and only had to ensure that the testcases still works despite the change in name. - Implemented
delMeetingcommand. #129 This task implements a new command as part of the CRUD features for meetings which was manageable. - Implemented tests for
delMeetingcommand. #140 To ensure thedelMeetingcommand works as intended, I wrote some test cases to ensure it works as intended. - Changed Client’s attributes
meetingto meeting listList<Meeting>. #164 This task was challenging as our team wanted to make a bidirectional association immutable and I spent hours trying to find a solution that can work. However, we were not able to find a solution and due to time, we settle with our current implementation of having the meeting list in client mutable. - Implemented
listClientby birthday to list client with upcoming birthday. #202 This task was manageable as this was an implementation that builds on functions we already had and I felt that this was similar to implementing a new feature. - Fixed valid date accepted by both the
MeetingDateandBirthday. #294 This task was fairly simple as I just added checks to ensure valid results.
- Refactored most
- Contributions to team-based task:
- Designed the User Interface for MyInsuRec. Mockup
- Find out that bidirectional association between 2 objects were hard to implement with immutability, ie
Meetingcontains aClientand aClientcontains a list ofMeeting.
- Documentation:
- README:
- Update the Target Users. #47
- User Guide:
- Added documentation for the features
addMeetinganddelMeeting. #57 and #58 - Added documentation for command feature. #59
- Updated documentation for the features
addClientanddelClient. #198 - Updated documentation for the feature
delMeeting. #198 - Added a
How to use this User Guidesection. #270 - Updated
Overviewsection by highlighting the keywords. #270 - Added
User Interfacesection with a Word Document to enable changes. #270
- Added documentation for the features
- Developer Guide:
- README:
- Notable Review Contributions:
- Suggested to use
OptionalforBirthdayattribute in client, this helps to prevent any possibleNullPointerExceptionerror to be thrown. #168 - Gave a comprehensive UG comment in order to standardize the format used in User Guide by all team member such that the User Guide seems like it is done by one person. #206
- Suggested to use