-
Notifications
You must be signed in to change notification settings - Fork 580
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-4] NUSCourseMates #25
Open
joel-foo
wants to merge
682
commits into
nus-cs2103-AY2324S1:master
Choose a base branch
from
AY2324S1-CS2103T-T17-4: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-T17-4] NUSCourseMates #25
joel-foo
wants to merge
682
commits into
nus-cs2103-AY2324S1:master
from
AY2324S1-CS2103T-T17-4: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
Chrainx
added a commit
to Chrainx/tp
that referenced
this pull request
Oct 1, 2023
add user story to DG
Improve code quality
Add test cases
Fix wording in added enhancement in PPP
* master: Update sequence diagram image Fix wording in added enhancement in PPP Write more tests Improve code quality Update DG Improve code quality Shorten PPP to fit the requirements
Change 'Address Book' to NUSCoursemates for standardization
Fix sort name - updated
* master: Resize Sort DG images Change 'Address Book' to NUSCoursemates for standardization Update UG for sort name Modify sort name method Update DG PlantUML Activity diagram for Sort Remove unnecessary string Remove unnecessary methods Update UG Add test cases for sort name class Fix sorting bug Add assertion
Update DG - Add manual testing
UG: - Update all the images to new header - Amend table in Add feature DG: - Add Theme Feature implementation
Add test cases
Update UG and DG
Update planned enhancement
…into branch-update-dg-final * 'master' of https://github.com/AY2324S1-CS2103T-T17-4/tp: Update DeveloperGuide.md Update PPP Fix wording Update planned enhancement # Conflicts: # docs/DeveloperGuide.md
Fix formatting issues
Fix formatting issues DG
Update PPP
Add error tables
Update images to include annotation
Add more error tables to various commands
* 'master' of https://github.com/AY2324S1-CS2103T-T17-4/tp: Add more error tables for the various commands
Fix hyperlinks
Add missing fullstops
Update UG -Sort
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.
Students in SoC often face the challenge of identifying fellow classmates who share their course schedules. The address book is designed to alleviate this difficulty by helping them connect with peers who are taking the same courses, which can be a particularly valuable resource when assessments involve group work.