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.

Sign up for GitHub

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

Jump to bottom

Refactor save command #217

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

Refactor save command #217

wants to merge 337 commits into from

Conversation

KhooDesmond
Copy link

No description provided.

annabeleng and others added 30 commits October 19, 2016 10:50
@annabeleng
Merge pull request #22 from CS2103AUG2016-T13-C3/update-developerguide
fb617ec 
Update DeveloperGuide
@KhooDesmond
Update FindCommand to find keywords in all event, deadlines and float…
4246a13 
…ing task panels
@KhooDesmond
Revert "Update FindCommand to find keywords in all event, deadlines a…
802dccb 
…nd floating task panels"

This reverts commit 4246a13.
@KhooDesmond
Merge branch 'master' of https://github.com/CS2103AUG2016-T13-C3/main
2caf1d8 
 floating task panels"

This reverts commit 4246a13.
@KhooDesmond
Update FindCommand to find keywords in all event, deadlines and float…
cb683f2 
…ing task panels
@shusiner
change some naming issues
4dffa0e
@annabeleng
Update UserGuide
814ac5d 
Modify add and edit command documentation
@annabeleng
Merge pull request #25 from CS2103AUG2016-T13-C3/update-userguide-add…
b388fd2 
…-edit-command

Update UserGuide
@kwonn
Merge pull request #23 from CS2103AUG2016-T13-C3/sort_deadline_and_ev…
4df7de7 
…ents

Sort deadline and events
@annabeleng
Update link to UI image
f68b40b
@KhooDesmond
Enhance find command to find only floating task, deadline or schedule
8e127fb
@KhooDesmond
Fix bug that displays wrong find result
804e17e
@KhooDesmond
Merge branch 'master' of https://github.com/CS2103AUG2016-T13-C3/main
89889ec
@kwonn
Update Parser to send the correct parameters to EditCommand
37b11d6
@KhooDesmond
Refactor all tests to match latest model
462f384
@KhooDesmond
Fixed error in DeadlineListPanel
fdebb77
@KhooDesmond
Support deletion of events and deadlines
121cfa2
@kwonn
Merge pull request #26 from CS2103AUG2016-T13-C3/modify_edit_function
f6ba06a 
Modify edit function
@kwonn
Merge branch 'find_command'
7f66c81 
# Conflicts:
#	src/main/java/seedu/malitio/commons/core/Messages.java
@kwonn
Merge branch 'delete_command'
873e58f 
# Conflicts:
#	src/main/java/seedu/malitio/model/Malitio.java
@kwonn
Fix some minor errors caused by merge.
96c5ef0
@kwonn
Support listing a specific type of task
4d819d5
@kwonn
Fix delete function
9248b45
@kwonn
Merge pull request #30 from CS2103AUG2016-T13-C3/list_command
dd2e126 
Support listing a specific type of task
@kwonn
Fix error in test case
de1563a
@KhooDesmond
Update Storage API documentation, fixes #18
a7d378c
@KhooDesmond
Make sure events start time is before end time
48ccc91
@KhooDesmond
Fix some errors, ensure editted events have start time before end time
f0d3821
@KhooDesmond
Merge pull request #34 from CS2103AUG2016-T13-C3/ensure_valid_events
336401b 
Ensure valid events (fixes #33)
@KhooDesmond
Fixed bugs in delete command
9fbfe33
KhooDesmond and others added 28 commits November 3, 2016 14:59
@KhooDesmond
Fix panel title font style
82220b4
@KhooDesmond
Fix ellipsis of index problem
3189111
@KhooDesmond
Merge pull request #82 from CS2103AUG2016-T13-C3/improve_ui_and_errors
4e7eb1c 
Improve ui and errors
@annabeleng
edit parser to parse input correctly by ignoring suffix of keywords.
ae3c82b 
Eg. it will not capture the "end" in friend as a keyword for an end date.
@annabeleng
Merge pull request #83 from CS2103AUG2016-T13-C3/edit_parser_to_check…
37e1a10 
…_for_task_keywords_properly

edit parser to parse input correctly by ignoring suffix of keywords.
@shusiner
updated grayout events that are overdue
a039cf8
@annabeleng
UI will scroll to the task which is changed/added
4e99996
@shusiner
change to (un)completing (un)completed task message.
26f23f2
@annabeleng
Merge pull request #84 from CS2103AUG2016-T13-C3/UI_scroll_to_new_or_…
5dd5051 
…edited_task

UI will scroll to the task which is changed/added
@annabeleng
Refactor EditCommand
ce86120 
Refactor EditCommandTest
@KhooDesmond
display day of week, fix find command to accommodate this change
3a76457
@shusiner
Update README.md
3f007bc
@annabeleng
Add header comments for methods in some classes.
325b796 
Modify format to follow java coding standard
Add collation tags "//@@author A0129595N" for collation
@annabeleng
Merge pull request #85 from CS2103AUG2016-T13-C3/refactor_edit_comman…
b99c805 
…d_and_test

Refactoring & ensure java coding standards are followed
@annabeleng
Update UserGuide.md
1309833 
Edit add and edit command
@annabeleng
Update UserGuide.md
0748160 
Add the restriction on task name
@annabeleng
Merge pull request #87 from CS2103AUG2016-T13-C3/annabeleng-patch-1
604deb7 
Update UserGuide.md
@annabeleng
Update UserGuide.md
983d805
@annabeleng
Merge pull request #88 from CS2103AUG2016-T13-C3/annabeleng-patch-2
3295b65 
Update UserGuide.md
@annabeleng
Update UserGuide.md
4f7b479
@annabeleng
Merge pull request #89 from CS2103AUG2016-T13-C3/annabeleng-patch-3
aff29ad 
Update UserGuide.md
@annabeleng
Update DeveloperGuide.md
3396132
@annabeleng
Remove browser icon from architecture
1c171ba
@annabeleng
Merge pull request #90 from CS2103AUG2016-T13-C3/annabeleng-patch-4
37fb6ab 
Update DeveloperGuide.md
@KhooDesmond
Fix bug with parsing, in particular parsing name from events and dead…
5e9efc5 
…lines that has date format 11-Nov-2016, 22:00 (Fri)
@KhooDesmond
Merge branch 'display_date_format'
6252717 
# Conflicts:
#	src/main/java/seedu/malitio/logic/parser/Parser.java
@KhooDesmond
Fix add deadline bug
000e684
@KhooDesmond
refactor save command
d1df69f
@nus-se-pr-bot
Copy link

Hi @KhooDesmond, your pull request title is invalid. It should be in the format of [Activity ID][Team ID] Your name, where [Activity Id] has no dashes or spaces (e.g. [T2A3] stands for Tutorial 2 Activity 3) and [Team ID] has one dash only and no spaces (e.g. [W14-A2] means Wednesday 2pm (14 hrs), Phase A, Team 2). Please follow the instructions given strictly and edit your title for reprocessing.

Submit only one activity per pull request (unless otherwise stated in instructions) and do remember to create your branches from the commit where the master branch is pointing at so that each PR you submit only consist of commits meant for the activity.

Note: this comment is posted by a bot. If you believe this is done in error, please create an issue at cs2103-pr-bot and add a link to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
FormatCheckRequested
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@KhooDesmond @nus-se-pr-bot @kwonn @annabeleng @shusiner