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-T17-1] TeamBuilder #78

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

Conversation

ChickenChiang
Copy link

TeamBuilder is a contact list that helps students and professionals to find people in their contacts to build teams with, for competitions and projects alike. TeamBuilder can organise contacts and add flags such that the team formation process is much faster

@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Patch coverage: 75.99% and project coverage change: +0.18 🎉

Comparison is base (928f146) 72.15% compared to head (069210e) 72.34%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #78      +/-   ##
============================================
+ Coverage     72.15%   72.34%   +0.18%     
- Complexity      399      606     +207     
============================================
  Files            70       95      +25     
  Lines          1232     1985     +753     
  Branches        125      229     +104     
============================================
+ Hits            889     1436     +547     
- Misses          311      459     +148     
- Partials         32       90      +58     
Impacted Files Coverage Δ
src/main/java/teambuilder/AppParameters.java 75.00% <ø> (ø)
src/main/java/teambuilder/Main.java 0.00% <ø> (ø)
src/main/java/teambuilder/MainApp.java 0.00% <0.00%> (ø)
src/main/java/teambuilder/commons/core/Config.java 68.18% <ø> (ø)
...ain/java/teambuilder/commons/core/GuiSettings.java 69.23% <ø> (ø)
...main/java/teambuilder/commons/core/LogsCenter.java 78.37% <ø> (ø)
...c/main/java/teambuilder/commons/core/Messages.java 0.00% <ø> (ø)
...rc/main/java/teambuilder/commons/core/Version.java 82.50% <ø> (ø)
...ain/java/teambuilder/commons/core/index/Index.java 100.00% <ø> (ø)
...er/commons/exceptions/DataConversionException.java 100.00% <ø> (ø)
... and 85 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

ChickenChiang and others added 26 commits March 27, 2023 17:38
AddCommandParser now does not throw exception if the major prefix is missing.
UnqieteamList class directly mirrors the behaviour of UniquePersonList. Likewise, the exceptions thrown are just the mirrored exceptions for persons.
…amBuilder

To maintain a UniqueTeamList in the TeamBuidler model, methods are created to maintain the UniqueTeamList
This method is needed for TeamBuilder model to check if there are duplicate teams. Teams are considered duplicates if they have the same name.
Added methods for maintaining Team list
Teamlist and associated commands
# Conflicts: Removed team tags on the sample data

#	src/main/java/teambuilder/model/util/SampleDataUtil.java
# Conflicts:
#	src/main/java/teambuilder/model/Model.java
#	src/main/java/teambuilder/model/ModelManager.java
#	src/main/java/teambuilder/model/TeamBuilder.java
#	src/test/java/teambuilder/logic/commands/AddCommandTest.java
#	src/test/java/teambuilder/logic/commands/RedoCommandIntegrationTest.java
#	src/test/java/teambuilder/logic/commands/UndoCommandTest.java
#	src/test/java/teambuilder/model/TeamBuilderTest.java
…into Merge

# Conflicts:
#	src/main/java/teambuilder/model/util/SampleDataUtil.java
# Conflicts:
#	src/main/java/teambuilder/model/Model.java
#	src/main/java/teambuilder/model/ModelManager.java
#	src/main/java/teambuilder/model/TeamBuilder.java
#	src/main/java/teambuilder/model/team/UniqueTeamList.java
#	src/main/java/teambuilder/model/team/exceptions/TeamNotFoundException.java
#	src/test/java/teambuilder/logic/commands/AddCommandTest.java
#	src/test/java/teambuilder/logic/commands/RedoCommandIntegrationTest.java
#	src/test/java/teambuilder/logic/commands/UndoCommandTest.java
#	src/test/java/teambuilder/model/TeamBuilderTest.java
Ensure that TeamName is converted to lower case before applying check
TeamName and person Name should both be equivalent regardless of casing and trailing white spaces
chen-jerry-junior and others added 30 commits April 10, 2023 21:46
- Specification for sort command
- Change screen shots for UG
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.

4 participants