forked from se-edu/addressbook-level4
-
Notifications
You must be signed in to change notification settings - Fork 64
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
[T13-1]Tutor Pal #57
Open
hrxiong
wants to merge
634
commits into
nus-cs2103-AY1819S1:master
Choose a base branch
from
CS2103-AY1819S1-T13-1: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
[T13-1]Tutor Pal #57
hrxiong
wants to merge
634
commits into
nus-cs2103-AY1819S1:master
from
CS2103-AY1819S1-T13-1: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
xhsien
approved these changes
Sep 23, 2018
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work :)
Good work. Some comments I have:
|
Also, add |
bannified
pushed a commit
to bannified/addressbook-level4
that referenced
this pull request
Oct 21, 2018
change file path of wishbook in UserPrefs class
Remember to add relevant diagrams (activity/class/object/sequence) in your developer guide to apply what we've learnt in class! |
Bellaaarh
pushed a commit
to Bellaaarh/addressbook-level4
that referenced
this pull request
Oct 29, 2018
…shaw_yeong_changes Update AnalyticsCommand, FinancialDatabaseParser and Deadline classes
pangjiahao
added a commit
to pangjiahao/cs2103-u-schedule
that referenced
this pull request
Oct 29, 2018
* Add a prototype calendar, using jfxtras library CalendarDisplay: The new class that extends UiPart add the calendarDisplay to the MainWIndow * merge calendarDisplay UI into existing ui rename CalendarEvent.getDescription to CalendarEvent.getDescriptionObject add functionality so ui will update when user updates calendarEvents fix calendar losing focus on toggling view
update commands for consistency and DeleteTimeCommand
UserGuide.adoc: add more detail about grade attribute
fix invalid addTime display error
# Conflicts: # src/test/java/seedu/address/logic/commands/TimeAddCommandTest.java # src/test/java/systemtests/AddTimeCommandTest.java # src/test/java/systemtests/TimeAddCommandTest.java
# Conflicts: # src/test/java/seedu/address/logic/commands/AddTimeCommandTest.java # src/test/java/systemtests/AddTimeCommandTest.java # src/test/java/systemtests/TimeAddCommandTest.java
update aboutus
no message
updated documentation
update PPP and developerGuide
update PPP
Update userguide
update developerguide
update developerGuide
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.
No description provided.