-
Notifications
You must be signed in to change notification settings - Fork 325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CS2103-W15-3] Teaching Assistant Contact Helper (TACH) #82
Open
jaysmyname
wants to merge
475
commits into
nus-cs2103-AY2122S2:master
Choose a base branch
from
AY2122S2-CS2103T-W15-3:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[CS2103-W15-3] Teaching Assistant Contact Helper (TACH) #82
jaysmyname
wants to merge
475
commits into
nus-cs2103-AY2122S2:master
from
AY2122S2-CS2103T-W15-3:master
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## master #82 +/- ##
============================================
+ Coverage 72.15% 72.31% +0.15%
- Complexity 399 504 +105
============================================
Files 70 79 +9
Lines 1232 1557 +325
Branches 125 185 +60
============================================
+ Hits 889 1126 +237
- Misses 311 375 +64
- Partials 32 56 +24
Continue to review full report at Codecov.
|
* 'master' of https://github.com/vanessaxuuan/tp: (45 commits) Update DG and UG Add AddTutorialGroupDescriptor diagram Add sequence diagram for addtg feature Update rationale Update DG with description of `addtg` command Add user stories and use cases Update uml diagrams with new sizes Update DG and UG Update dg Add design considerations Add deletetg implementation description Tweak invalid preamble and invalid value cases for deletetg parser test Resolve requested changes by adding clarity Replace addressbook mentions with TACH Replace addressbook mentions with TACH Remove DeleteTutorialGroupDescriptor Remove unused logos Dummy commit update use case Update dg with implementation details of GitHub and Telegram ...
Change find command such that it will - Find and list all Students in address book whose name contains ALL of the argument keywords. - instead of ANY of the argument keywords. update MESSAGE_ADD_TUTORIAL_GROUP_SUCCESS output for addTutorialGroupCommand.
Allow text wrap
Allow text wrap for telegram, github and email labels
Add padding between vbox
Update edit
# Conflicts: # src/test/java/seedu/address/logic/commands/CommandTestUtil.java
Delete tg from all students feature
LGTM Resolve #80
Add test cases to check for empty input to FindCommand - isolated from FindCommandParser
Update find name feature LGTM Resolve #78
Enable assertions
Update UG LGTM
Update ppp, delete the example ppp
Fix uml diagrams LGTM
Edit help guide LGTM
* 'master' of https://github.com/vanessaxuuan/tp: edit helpguide Update add student sequence diagram image Resolve add student sequence diagram Update ppp, delete the example ppp Reload UML diagrams Update UML diagrams and fix grammatical errors in ug no message no message edit help guide # Conflicts: # docs/diagrams/AddTutorialGroupSequenceDiagram.puml # docs/images/AddTutorialGroupSequenceDiagram.png
Update v1.4 documentation LGTM
Change version number to v1.4
…-storage-file-name Revert "Refactor addressbook json files to TACH"
Refactor TACH.json to addressbook.json and fix some errors in ug and dg LGTM
Update AboutUs
Update ppp LGTM
Update index.md LGTM
Update ugdg intro
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
TACH helps CS Teaching Assistants tutoring multiple modules & classes by keeping track of their students and monitoring their progress on their tutorials. It is optimized for CLI users so that frequent tasks can be done faster by typing in commands.