Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
matthewyip1511 created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
Same for here!
matthewyip1511 created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
Looks like this method is not tested, maybe we can add some tests for here 👍
matthewyip1511 created a review on a pull request on AY2425S1-CS2103-F12-1/tp
LGTM, other than for comments stated 🙂
matthewyip1511 created a review on a pull request on AY2425S1-CS2103-F12-1/tp
LGTM, other than for comments stated 🙂
github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103-F12-1/tp
- deploy: 636af09ba26d1d8df332030aecdf55d9991c32fc d5c60ab
matthewyip1511 pushed 5 commits to master AY2425S1-CS2103-F12-1/tp
- Add DeleteInsuranceCommandParserTest.java to increase code coverage for DeleteInsuranceCommandParser 9985505
- Fix checkstyle error 2728112
- Fix checkstyle error fc95c86
- Fix checkstyle e4bc8fa
- Merge pull request #136 from yanqiyqh/deleteinsurancecommandparser-test Add DeleteInsuranceCommandParserTest.java 636af09
matthewyip1511 closed a pull request on AY2425S1-CS2103-F12-1/tp
Add DeleteInsuranceCommandParserTest.java
Currently, we have no tests for DeleteInsuranceCommandParser. Let's add tests for DeleteInsuranceCommandParser to increase our test coverage.
codecov[bot] created a comment on a pull request on AY2425S1-CS2103-F12-1/tp
## [Codecov](https://app.codecov.io/gh/AY2425S1-CS2103-F12-1/tp/pull/136?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...
codecov[bot] created a comment on a pull request on AY2425S1-CS2103-F12-1/tp
## [Codecov](https://app.codecov.io/gh/AY2425S1-CS2103-F12-1/tp/pull/135?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...
yanqiyqh opened a pull request on AY2425S1-CS2103-F12-1/tp
Add DeleteInsuranceCommandParserTest.java
Currently, we have no tests for DeleteInsuranceCommandParser. Let's add tests for DeleteInsuranceCommandParser to increase our test coverage.
yanqiyqh opened a pull request on AY2425S1-CS2103-F12-1/tp
DeleteClaimCommandParserTest
Currently, we have no tests for DeleteClaimCommandParser. Let's add tests for DeleteClaimCommandParser to increase our test coverage.
github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103-F12-1/tp
- deploy: aa870191dfa757ce5e950b5726d6ed1afc53afbf bd3cb11
matthewyip1511 pushed 5 commits to master AY2425S1-CS2103-F12-1/tp
- Fix equals for all new classes - The equals methods has to change as insurance plans now have claims in them as well... f2ef08c
- Merge branch 'master' into rezwan-fix-equals 24cf8f9
- Remove problematic code This modification to equals method of client is causing alot of issues in the code. Let's r... 8a301ef
- Fix nits for PR#129 Changed the equals method in Insurance Plan manager to fit nicely with the rest of the equals me... 9d9bbf0
- Merge pull request #129 from RezwanAhmed123/rezwan-fix-equals Fix Equals Method for New Classes aa87019
matthewyip1511 closed a pull request on AY2425S1-CS2103-F12-1/tp
Fix Equals Method for New Classes
Equals methods for Insurance Plans Manager, Insurance Plan and claims were updated. Client equals method was left alone as it raised too many issues when modified.
github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103-F12-1/tp
- deploy: 9b4874bd174749b97c199856ecae6746419b4832 8ca1fc9
RezwanAhmed123 closed an issue on AY2425S1-CS2103-F12-1/tp
Create a CloseClaimsParser object
To parse inputs from the user for closing of claims. Ensure that the claim that the user wants to close exists in the first place AND has not been closed yet.
RezwanAhmed123 closed an issue on AY2425S1-CS2103-F12-1/tp
Create a closeClaims command
follow similar to mark/unmark inside of IP.
RezwanAhmed123 pushed 11 commits to master AY2425S1-CS2103-F12-1/tp
- Add baseline for CloseClaimCommand 2e1bb5e
- Merge branch 'master' into matthew-add-closeClaim-command 6b8ef09
- Add main files for CloseClaimCommand Most of the previous commits were work-in-progress and done as I worked on the f... ab301e1
- Fix typo in CloseClaimCommandParser c6f323c
- Undo changes in userguide There will be another PR opened soon to make some changes in the userguide Hence, all chang... d93ea6c
- Fix grammer and rename a variable f8838c2
- Fix typo 44eeb65
- Fix typo 82629f5
- Fix merge conflict aaf284e
- Fix merge conflict 6961913
- Merge pull request #130 from matthewyip1511/matthew-add-closeClaim-command Add CloseClaimCommand and CloseClaimComma... 9b4874b
RezwanAhmed123 closed a pull request on AY2425S1-CS2103-F12-1/tp
Add CloseClaimCommand and CloseClaimCommandParser
Adding the `CloseClaimCommand` and `CloseClaimCommandParser` to support the close claim feature as listed in #85 and #88. Currently the test coverage is `0%`, but this will be fixed by the tests...
RezwanAhmed123 created a comment on a pull request on AY2425S1-CS2103-F12-1/tp
Will merge as tests will be written later.
github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103-F12-1/tp
- deploy: aee4139514ff060ce7dffdfc2ed2db9943b3801e dcf55a5
RezwanAhmed123 closed an issue on AY2425S1-CS2103-F12-1/tp
Update UserGuide for v1.4 features
Also to fix bugs identified along the way
RezwanAhmed123 pushed 4 commits to master AY2425S1-CS2103-F12-1/tp
- Add line break fb439ca
- Merge branch 'master' of https://github.com/matthewyip1511/tp 19391e4
- Update userguide Adds userguide entry for ClosClaimCommand Also fixes some bugs found after the last iteration in v1.3 9a32d57
- Merge pull request #132 from matthewyip1511/matthew-update-userguide Update UserGuide for v1.4 aee4139