Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
blanklogic created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Overall LGTM! Thanks for updating the Ui.png from the mock up to our actual GUI changes.
codecov[bot] created a comment on a pull request on AY2425S1-CS2103T-T09-1/tp
## [Codecov](https://app.codecov.io/gh/AY2425S1-CS2103T-T09-1/tp/pull/72?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...
uniqly opened a pull request on AY2425S1-CS2103T-T09-1/tp
Update user guide with new features
* Remove mentions of AB3 * Add `apply` and `withdraw` to features section
uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
See my comment on `Messages.java`
uniqly created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
This is duplicated, the status was already added in lines 46 & 47
codecov[bot] created a comment on a pull request on AY2425S1-CS2103T-T09-1/tp
## [Codecov](https://app.codecov.io/gh/AY2425S1-CS2103T-T09-1/tp/pull/71?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...
uniqly opened a pull request on AY2425S1-CS2103T-T09-1/tp
Improved handling of optional inputs
Resolves #65 The app now has a way to handle optional phone number and address without resorting to a reserved string.
uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM. Screenshots would be appreciated to see how the UI looks like after the changes. 😄
meerkatboy closed a pull request on AY2425S1-CS2103T-T09-1/tp
Edited edit and find commands
Changes to ensure application works with companies instead of persons Closed, see #69
codecov[bot] created a comment on a pull request on AY2425S1-CS2103T-T09-1/tp
## [Codecov](https://app.codecov.io/gh/AY2425S1-CS2103T-T09-1/tp/pull/69?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...
meerkatboy opened a pull request on AY2425S1-CS2103T-T09-1/tp
Branch add applications to edit command
Added methods to editCommand class to return new Company with edited status when application is added
uniqly created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
This shouldn't be here as it is no longer used.
uniqly created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
Should be `Model` instead of `ModelCompany`
uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
I think `EditCompanyDescriptor` should include applications as it will be used for the `apply` and `withdraw` applications commands
uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
I think `EditCompanyDescriptor` should include applications as it will be used for the `apply` and `withdraw` applications commands
blanklogic reopened a pull request on AY2425S1-CS2103T-T09-1/tp
Add initial implementation of Apply Command
Resolves #60 and #63 ## Changes - Initial Implementation of Apply Command using ```java apply INDEX n/NAME d/DESCRIPTION apply 1 n/Full Stack Engineer d/Requires ReactJS and ExpressJS ``` ...
blanklogic closed a pull request on AY2425S1-CS2103T-T09-1/tp
Add initial implementation of Apply Command
Resolves #60 and #63 ## Changes - Initial Implementation of Apply Command using ```java apply INDEX n/NAME d/DESCRIPTION apply 1 n/Full Stack Engineer d/Requires ReactJS and ExpressJS ``` ...
codecov[bot] created a comment on a pull request on AY2425S1-CS2103T-T09-1/tp
## [Codecov](https://app.codecov.io/gh/AY2425S1-CS2103T-T09-1/tp/pull/68?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...
realqijun opened a pull request on AY2425S1-CS2103T-T09-1/tp
Add GUI for Applications + handle optional values
github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103T-T09-1/tp
- deploy: bdef4853e449322beb0f22804419bfc83d1f4481 050d99b
Shux347 pushed 7 commits to master AY2425S1-CS2103T-T09-1/tp
- Add Application list to Company 79eb195
- Update storage structure eae129a
- Integrate new storage edc8f7f
- Update test cases for new storage dfe3c1e
- Remove extra whitespace in Company.java c2824d3
- Fix missing test coverage for Company#equals() cc1eaa4
- Merge pull request #67 from uniqly/57-add-application-list-into-company-class Add application list to `Company` and ... bdef485
Shux347 closed a pull request on AY2425S1-CS2103T-T09-1/tp
Add application list to `Company` and update storage
Resolves #57 and #64
blanklogic created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Please remove PersonListPanel.java from the PR and properly sync your fork as it seems like your fork repo has not been synced to upstream.