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

[T10-A2][W10-C3] #196

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

[T10-A2][W10-C3] #196

wants to merge 344 commits into from

Conversation

filbert-c
Copy link

ready for review

filbert-c and others added 30 commits October 14, 2016 14:43
@filbert-c
1. Resolved edit partial commmand test fail
6772209
@filbert-c
test to merge period time
8096759
TODO: test fixes
0375fc0
TODO: wierd bug to be fixed at the end of passing 98/98 tests
da3734b
@YesYouKenSpace
Merge branch 'master' of https://github.com/CS2103AUG2016-W10-C3/main
afc714e
Commented
66ce274
1.Optional added, Task now supports , Date , Date + Start Time, Date …
b993feb 
…+ end time, No date.

2. TODO : fix tests
fixed all test cases.
ea673f1 
REQUESTING HELP: Unable to solve error at the end of clearing all test cases 98/98
Damith says " Not on FX Application Thread"
@YesYouKenSpace
Merge pull request #33 from CS2103AUG2016-W10-C3/PeriodV2
0e7cd69 
Merging Period into Time for 1 class
@YesYouKenSpace
added additional testcase and temporary fix for it
67d6d4c
@filbert-c
Merge pull request #34 from CS2103AUG2016-W10-C3/PeriodV2_fixForEdit
1dcd533 
added additional testcase and temporary fix for it
@YesYouKenSpace
Stricter testcase
d7d5696
@YesYouKenSpace
Merge pull request #35 from CS2103AUG2016-W10-C3/PeriodV2_fixForEdit
906d9cd 
Stricter testcase
@YesYouKenSpace
Merge pull request #36 from CS2103AUG2016-W10-C3/PeriodV2
398703f 
Period v2
@YesYouKenSpace
created SortedObservableArrayList
24d31cf
@YesYouKenSpace
From hereon, all Tasks shall be inserted in order and the data shall …
806f8a5 
…always be sorted. No unsorted data SHALL PASS!
@YesYouKenSpace
UnmodifiableObservableList uses SortedObservableArrayList too. Implem…
0065ab2 
…ented SortedObservableArrayList.equals(). Some Refactoring
@YesYouKenSpace
Modified TestUtil.addTasksToList to add in order
58742a8
@MCHowl
Fix errors in DeveloperGuide.md
462645a
@MCHowl
Merge branch 'master' of https://github.com/CS2103AUG2016-W10-C3/main
95450ef 
Conflicts:
	docs/DeveloperGuide.md
@MCHowl
Update DeveloperGuide.doc to match changes in markdown version
dbdc43d
@YesYouKenSpace
Merge pull request #37 from CS2103AUG2016-W10-C3/issue25_usingSortedList
a17ce1d 
Sorted Task Data Structure
@MCHowl
Enable parsing of arugments into ListCommand
64cd0f3 
No functionality
@MCHowl
Implement functionality for "list completed"
4883408
@MCHowl
Implemented functionality to list upcoming and overdue task
2aabe77
@MCHowl
Fixed test cases
21a534c
@MCHowl
Merge branch 'extend-list-command'
0f16aa5 
Close #19
@MCHowl
Fix missing lines in testTask.java
4c4e942
@YesYouKenSpace
Remove println in TaskListPanelHandle.java
33280f6 
Trivial change. Will merge immediately as it does not affect anything.
@filbert-c
Added Mark Command
43444ea
filbert-c and others added 30 commits November 6, 2016 22:50
@filbert-c
updated docs
ce1ffda
@YesYouKenSpace
no idead why this broke but fixed it
29633c2
@filbert-c
Merge branch 'master' of https://github.com/CS2103AUG2016-W10-C3/main
1e93f0c
@YesYouKenSpace
try non-headless test
34640d7
Merge pull request #95 from CS2103AUG2016-W10-C3/bugFix_UndoEdit
af7c465 
Bug fix undo edit
@YesYouKenSpace
Increase test coverage for Edit
19e6289
@YesYouKenSpace
Updated Collated
fc82c94
@YesYouKenSpace
tried to improve code quality
7e9db6e
@YesYouKenSpace
Documentation submission
ce132f3
Merge pull request #96 from CS2103AUG2016-W10-C3/improving_test_coverage
e9e81da 
Improving test coverage
@YesYouKenSpace
Correct location of ManualTesting
54312ca
Update TestScript.md
f26001e
@filbert-c
Remove commented unused code
a56b5ae
@filbert-c
Revert "Remove commented unused code"
fde255d 
This reverts commit a56b5ae.
@filbert-c
remove unused code
edb8932
@filbert-c
remove unused code
ded60a5
Merge pull request #97 from CS2103AUG2016-W10-C3/removeUnusedCode
be89c29 
Remove unused code
added test data
c8fac8e
Rename SampleDate.xml to SampleData.xml
172e5cf
Merge pull request #98 from CS2103AUG2016-W10-C3/Testdata
2de297f 
added test data
@YesYouKenSpace
Improve test coverage for SortedObservableList.java
9f26815
@YesYouKenSpace
recompile collate
c244314
Merge pull request #99 from CS2103AUG2016-W10-C3/improving_test_coverage
b337ade 
Improve test coverage for SortedObservableList.java
@YesYouKenSpace
Changed screenshot reupdated docs
d74265e
Merge pull request #100 from CS2103AUG2016-W10-C3/bugFix_doc
dceb328 
Changed screenshot reupdated docs
Delete ~$10-C3][SmartyDo].doc
48034b9
Delete ~$erGuide.docx
62f9dd2
Delete ~WRL0003.tmp
7594bc1
@YesYouKenSpace
recollate
a95a8ec
@YesYouKenSpace
Reduced test cases
dd8b8e5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damithc damithc damithc left review comments

Assignees
No one assigned
Labels
FormatCheckRequested
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@filbert-c @nus-se-pr-bot @damithc @MCHowl @YesYouKenSpace @MoonByunghun