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
person
andaddressbook
toclient
and 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
delMeeting
command. #129 This task implements a new command as part of the CRUD features for meetings which was manageable. - Implemented tests for
delMeeting
command. #140 To ensure thedelMeeting
command works as intended, I wrote some test cases to ensure it works as intended. - Changed Client’s attributes
meeting
to 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
listClient
by 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
MeetingDate
andBirthday
. #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
Meeting
contains aClient
and aClient
contains a list ofMeeting
.
- Documentation:
- README:
- Update the Target Users. #47
- User Guide:
- Added documentation for the features
addMeeting
anddelMeeting
. #57 and #58 - Added documentation for command feature. #59
- Updated documentation for the features
addClient
anddelClient
. #198 - Updated documentation for the feature
delMeeting
. #198 - Added a
How to use this User Guide
section. #270 - Updated
Overview
section by highlighting the keywords. #270 - Added
User Interface
section with a Word Document to enable changes. #270
- Added documentation for the features
- Developer Guide:
- README:
- Notable Review Contributions:
- Suggested to use
Optional
forBirthday
attribute in client, this helps to prevent any possibleNullPointerException
error 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