Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

AY2425S1-CS2103T-T09-1/tp

blanklogic closed an issue on AY2425S1-CS2103T-T09-1/tp
Add withdraw command
blanklogic closed a pull request on AY2425S1-CS2103T-T09-1/tp
Add withdraw command
Closes #70
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/75?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...

View on GitHub

uniqly opened a pull request on AY2425S1-CS2103T-T09-1/tp
Fix typo in website logo png file
Resolves #74
uniqly opened an issue on AY2425S1-CS2103T-T09-1/tp
Fix website missing icon
uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM! 👍

View on GitHub

github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103T-T09-1/tp
  • deploy: 139979af86616a095f92971cda9f806f96a39447 e1035a3

View on GitHub

blanklogic pushed 7 commits to master AY2425S1-CS2103T-T09-1/tp
  • Update features in user guide ea403ab
  • Merge branch 'master' of https://github.com/AY2425S1-CS2103T-T09-1/tp into 55-update-user-guide c016720
  • Replace mentions of AB3 with InterBuddy in user guide 075330a
  • Include new features in user guide 8ed0edc
  • Remove AB3 mentions in index.md d046cbc
  • Merge branch 'master' of https://github.com/AY2425S1-CS2103T-T09-1/tp into 55-update-user-guide bb8c702
  • Merge pull request #72 from uniqly/55-update-user-guide Update user guide with new features 139979a

View on GitHub

blanklogic closed an issue on AY2425S1-CS2103T-T09-1/tp
Update user guide
Update user guide to match InternBuddy
blanklogic closed 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
blanklogic created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM!

View on GitHub

Shux347 closed an issue on AY2425S1-CS2103T-T09-1/tp
Add Company status change logic
When a new application is created for a company that is not "APPLIED" status, change status to "APPLIED"
Shux347 pushed 9 commits to master AY2425S1-CS2103T-T09-1/tp
  • Add initial implementation of Apply Command 83419ce
  • Add JUnit Test cases for new classes 8ce56d1
  • Add JUnit Tests for new Classes 78fb2dc
  • Resolve Merge Conflicts 0ecacf2
  • Add more JUnit Tests c978fb3
  • Merge branch 'master' into feat-add-app 023133d
  • Update Apply command from initial implementation 8998a63
  • Fix bug where App Status is not parsed 72b1ad3
  • Merge pull request #66 from blanklogic/feat-add-app Add initial implementation of Apply Command 699357e

View on GitHub

Shux347 closed an issue on AY2425S1-CS2103T-T09-1/tp
Allow users to add applications for each company
- Add a new command `apply INDEX d/description [s/status] [r/remarks]` - `status = Applied` by default (other status are `OA`, `Interview`, `Offered`, `Accepted`, `Rejected`
Shux347 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 ``` ...
blanklogic created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM! Great Test cases for ensuring good CI/CD.

View on GitHub

uniqly created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
After discussion, the current behaviour is fine. But I still suggest that `ParserUtil#parseIndexes` be renamed to `ParserUtil#parseWithdrawIndex` to avoid confusion.

View on GitHub

uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp

View on GitHub

uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM! 😄

View on GitHub

uniqly created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
I dont think this should be hard coded like that.

View on GitHub

uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp

View on GitHub

github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103T-T09-1/tp
  • deploy: 9852b7febc1fd55070fdbb35780730c030690a2d 12eef9c

View on GitHub

Shux347 pushed 4 commits to master AY2425S1-CS2103T-T09-1/tp
  • Update phone and address to handle optional inputs ed34927
  • Update tests for optional params 8ac3329
  • Hide internal value of address and phone 34e14f6
  • Merge pull request #71 from uniqly/65-gracefully-handle-optional-inputs Improved handling of optional inputs 9852b7f

View on GitHub

Shux347 closed an issue on AY2425S1-CS2103T-T09-1/tp
Gracefully handle optional inputs
Currently we are using a "hotfix" solution to handle optional inputs for the the `add` command. (See #58) For instance, if the user decides not to include a phone number in the `add` command, a ...
Shux347 closed 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.
Shux347 created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM

View on GitHub

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/73?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=A...

View on GitHub

realqijun created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
Typo here

View on GitHub

realqijun created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Minor mistake but LGTM! ``` ╱|、 (˚ˎ 。7 |、˜〵 じしˍ,)ノ ```

View on GitHub

realqijun created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Minor mistake but LGTM! ``` ╱|、 (˚ˎ 。7 |、˜〵 じしˍ,)ノ ```

View on GitHub

Load more