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

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

View on GitHub

blanklogic created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Overall LGTM!

View on GitHub

blanklogic created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
Might not be good to include comments of unused code in the PR

View on GitHub

blanklogic created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Minor nits, loved the star you have drawn. Small change, otherwise LGTM!

View on GitHub

blanklogic created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
Good write up in the User Guide. It's clear and succinct

View on GitHub

blanklogic created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Overall LGTM!

View on GitHub

blanklogic created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Overall LGTM!

View on GitHub

github-actions[bot] pushed 1 commit to gh-pages AY2425S1-CS2103T-T09-1/tp
  • deploy: 5f28e8fc457974b4bb392cbc6b348bef8b420e19 c6eeb92

View on GitHub

blanklogic closed an issue on AY2425S1-CS2103T-T09-1/tp
Add Apply and Update feature to Developer Guide
# Add Apply and Update feature to Developer Guide Building on top of #105 due: 24 Oct 2024 ## Changes * Add Apply feature implementation to Developer Guide * Add Update feature implementation...
blanklogic pushed 4 commits to master AY2425S1-CS2103T-T09-1/tp
  • Add Apply and Update to Developer Guide 3911811
  • Fix typos in Developer Guide 7a0719b
  • Update Apply and Update command sequence diagrams 1879a96
  • Merge pull request #113 from blanklogic/task-update-dg Add Apply and Update to Developer Guide 32f622e

View on GitHub

blanklogic closed a pull request on AY2425S1-CS2103T-T09-1/tp
Add Apply and Update to Developer Guide
Resolves #112 ## ApplyCommandSequence.puml ![ApplyCommandSequence](https://github.com/user-attachments/assets/0dd31366-e2a7-49bc-8b8e-d5207056c053) ## UpdateCommandSequence.puml ![UpdateCom...
blanklogic pushed 9 commits to master AY2425S1-CS2103T-T09-1/tp
  • Add initial implementation of Update command 79c512e
  • Merge branch 'master' into feat-edit-app 23feae9
  • Add JUnit tests for Update command 123f0c6
  • Update User Guide with update command implementation 8d12d60
  • Merge branch 'master' into feat-edit-app ce172e3
  • Fix renaming bug introduced in merge c210500
  • Add Prefixes for Update command cf768c3
  • Fix typo in User Guide c7fb806
  • Merge pull request #111 from blanklogic/feat-edit-app Add initial implementation of Update command 5f28e8f

View on GitHub

blanklogic closed an issue on AY2425S1-CS2103T-T09-1/tp
Add update command
# Add update command Format: `update COMPANY_INDEX APPLICATION_INDEX as/APPLICATION_STATUS` due: 24 Oct 2024
blanklogic closed a pull request on AY2425S1-CS2103T-T09-1/tp
Add initial implementation of Update command
Resolves #92
uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM. Some minor code quality improvements can be worked on.

View on GitHub

uniqly created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
I think this can be split into a few boolean expressions instead of one long expression (to be less confusing)

View on GitHub

uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM. Minor code quality improvements can be worked on.

View on GitHub

uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
LGTM. Minor code quality improvements can be worked on.

View on GitHub

uniqly created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
This does a decent job of capturing how the favourite command is supposed to function. Great job! 👍 I think some refinement could be done; such as starting the sequence from the user as an actor.

View on GitHub

realqijun created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
Interesting! Elegant and simple solution to find keywords!

View on GitHub

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

View on GitHub

realqijun created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Great job on the find function! Well tested too! LGTM ``` ████ ██ ██ ▓▓ ██ ...

View on GitHub

realqijun created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Great job on the find function! Well tested too! LGTM ``` ████ ██ ██ ▓▓ ██ ...

View on GitHub

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

View on GitHub

meerkatboy created a review on a pull request on AY2425S1-CS2103T-T09-1/tp
Looks good to me, good job Qi Jun!

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

View on GitHub

uniqly created a review comment on a pull request on AY2425S1-CS2103T-T09-1/tp
Minor typo. Should be `update c/1 app/1 as/OA` instead.

View on GitHub

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

View on GitHub

meerkatboy opened a pull request on AY2425S1-CS2103T-T09-1/tp
Expand find to include application and tags
Searching by company name only will not yield many functional results. Expanding the find command to include words in application description and tags can help users search for companies in the ...
blanklogic opened an issue on AY2425S1-CS2103T-T09-1/tp
Add Favourite feature to Developer Guide
due: 24 Oct 2024
Load more