Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
RezwanAhmed123 created a comment on an issue on AY2425S1-CS2103-F12-1/tp
looks mostly done, i think can close (96% is done anyways)
RezwanAhmed123 closed an issue on AY2425S1-CS2103-F12-1/tp
Update test for AddInsuranceCommand
Already done, but can be improved with the use of temporary data space.
RezwanAhmed123 opened a pull request on AY2425S1-CS2103-F12-1/tp
Improve test coverage for insurance plans manager
The test coverage for insurance plans manager object is quite low. Lets add more tests to increase the code coverage and look for any bugs.
RezwanAhmed123 created a comment on an issue on AY2425S1-CS2103-F12-1/tp
Seems like it has hit 100% due to #106
RezwanAhmed123 closed an issue on AY2425S1-CS2103-F12-1/tp
Improve test for ListClaimsCommand
Right now, there are some tests there but it is not 100% test coverage. Let's improve this to meet 100%.
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/143?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...
RezwanAhmed123 opened a pull request on AY2425S1-CS2103-F12-1/tp
Add auto sort
Currently, the claims are sorted in the order in which they are added. They should instead appear in lexicographical order which would be more appropriate. They should be sorted automatically once ...
zi-yii created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
We have to include full stop for java doc Example: ``` /** * Returns lateral location of the specified position. * If the position is unset, NaN is returned. * * @param x X coordinate ...
zi-yii created a review on a pull request on AY2425S1-CS2103-F12-1/tp
LGTM, just need to add full stop for java doc comments
zi-yii created a review on a pull request on AY2425S1-CS2103-F12-1/tp
LGTM, just need to add full stop for java doc comments
zi-yii created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
Yeps, it led to a bug where insurance plan is not detected although the insurance plan was already added to the client
yanqiyqh opened an issue on AY2425S1-CS2103-F12-1/tp
Create Updated Class Diagram for Person Class
**Description:** We need to create an updated class diagram for the `Person` class that illustrates its attributes and methods, as well as its relationships with the newly implemented `InsurancePl...
yanqiyqh opened an issue on AY2425S1-CS2103-F12-1/tp
Add UML Sequence Diagram for Add Insurance Plan Feature
**Description:** We have implemented the "Add Insurance Plan" feature, and it is important to update the developer guide to include comprehensive documentation. Specifically, we need to create and...
yanqiyqh opened an issue on AY2425S1-CS2103-F12-1/tp
Add UML Sequence Diagram for Add Claim Feature
**Description:** We have implemented the "Add Claim" feature, and it is important to update the developer guide to include comprehensive documentation. Specifically, we need to create and add a UM...
yanqiyqh opened an issue on AY2425S1-CS2103-F12-1/tp
Add UML Sequence Diagram for Close Claim Feature
**Description:** We have recently implemented the "Close Claim" feature, and it is essential to update the developer guide to include relevant documentation. Specifically, we need to add a UML seq...
RezwanAhmed123 closed an issue on AY2425S1-CS2103-F12-1/tp
Improve UI to match inSUREance app specs
Let's improve the UI to make it look nicer 🙂
github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103-F12-1/tp
- deploy: 5ac38762350ecd4ec8d223c75fc5d97bc3329d18 f701171
RezwanAhmed123 pushed 6 commits to master AY2425S1-CS2103-F12-1/tp
- Merge branch 'master' into ziyi-insurance-tests e35311f
- Create DeleteInsuranceCommandTest.java 37a74f0
- Add test for equals method for DeleteInsuranceCommandTest.java 6e1a927
- Add javadoc to DeleteInsuranceCommandTest.java 181f309
- Fix checkstyle errors for DeleteInsuranceCommandTest.java 6182cdd
- Merge pull request #138 from zi-yii/ziyi-insurance-tests 5ac3876
RezwanAhmed123 closed a pull request on AY2425S1-CS2103-F12-1/tp
Add DeleteInsuranceCommandTest.java
There are no tests for DeleteInsuranceCommand.java. Let's add tests for DeleteInsuranceCommand to increase test coverage.
RezwanAhmed123 created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
Yep seems like it!
RezwanAhmed123 created a review on a pull request on AY2425S1-CS2103-F12-1/tp
LGTM! Just need to fix some merge conflicts.
yanqiyqh created a review comment on a pull request on AY2425S1-CS2103-F12-1/tp
was this modified because the new equals() method in InsurancePlan tests for equality in claims too?