Skip to content
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-T14-1] EduConnect #28

Open
wants to merge 502 commits into
base: master
Choose a base branch
from

Conversation

fy17ohhh
Copy link

EduConnect helps an active TA, to manage contact details of students in both big or small tutorial classes, while also keeping track of student progress, accessing links to their projects, or simply de-conflict class schedules. It is optimized for CLI users to manage contacts efficiently by typing fast.

Copy link

codecov bot commented Feb 18, 2024

Codecov Report

Attention: Patch coverage is 77.81300% with 140 lines in your changes are missing coverage. Please review.

Project coverage is 75.41%. Comparing base (b896eb0) to head (74aa9e2).

❗ Current head 74aa9e2 differs from pull request most recent head 290eafe. Consider uploading reports for the commit 290eafe to get more accurate results

Files Patch % Lines
...in/java/educonnect/logic/commands/HelpCommand.java 37.93% 12 Missing and 6 partials ⚠️
.../main/java/educonnect/logic/parser/ParserUtil.java 74.07% 12 Missing and 2 partials ⚠️
src/main/java/educonnect/ui/StudentListPanel.java 0.00% 11 Missing ⚠️
src/main/java/educonnect/logic/Messages.java 47.05% 8 Missing and 1 partial ⚠️
...in/java/educonnect/logic/commands/CopyCommand.java 73.52% 7 Missing and 2 partials ⚠️
...n/java/educonnect/logic/commands/SlotsCommand.java 84.90% 3 Missing and 5 partials ⚠️
...dent/predicates/EmailMatchesKeywordsPredicate.java 27.27% 8 Missing ⚠️
...student/predicates/IdMatchesKeywordsPredicate.java 27.27% 8 Missing ⚠️
...t/predicates/TelegramMatchesKeywordsPredicate.java 27.27% 8 Missing ⚠️
...onnect/model/student/timetable/AvailableSlots.java 89.85% 3 Missing and 4 partials ⚠️
... and 18 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #28      +/-   ##
============================================
+ Coverage     75.26%   75.41%   +0.15%     
- Complexity      419      733     +314     
============================================
  Files            71       93      +22     
  Lines          1338     2384    +1046     
  Branches        126      296     +170     
============================================
+ Hits           1007     1798     +791     
- Misses          301      483     +182     
- Partials         30      103      +73     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

matthewken19 and others added 29 commits March 28, 2024 22:49
Update list command with option to list timetable
No support for parsing inputs to reflect finding slots functionality.

Having parser support allows for inputs from the CLI GUI.

Let's
* Modify CliSyntax to include additional prefixes
* Add SlotsCommand to handle the `slots` command
* Add SlotsCommandParser to handle parsing
* Remove Timetable overloaded constructors and have the parser handle optional inputs
* Update the TimetableTest accordingly
* Update ParserUtil to have additional parsing capabilities in parsing new type of String input
* Update AddressBookParser to have additional command
Includes:
* SlotsCommandTest
* SlotsCommandParserTest
* ParserUtilTest
Includes:
* SlotsCommandTest
* SlotsCommandParserTest
* ParserUtilTest
* Move the computation of finding common slots into Model and ModelManager
* Better design principle
* More closes adheres to Open-Close Principle and Law of Demeter
* Move the computation of finding common slots into Model and ModelManager
* Better design principle
* More closely adheres to Open-Close Principle and Law of Demeter
marclamp and others added 30 commits April 14, 2024 22:54
…ixErrorMessage

Branch update wrong timetable prefix error message
Regex bugfix for link. email, and telegram
Remove repeated information in notes about command format
Update DG and include copy description
Rename variable to comply convention
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants