forked from se-edu/addressbook-level4
-
Notifications
You must be signed in to change notification settings - Fork 69
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
[v1.0][W15-B1] E.T. (Employees Tracker) #43
Open
IzHoBX
wants to merge
874
commits into
nus-cs2103-AY1718S2:master
Choose a base branch
from
CS2103JAN2018-W15-B1: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.
+28,288
−2,084
Open
Changes from 93 commits
Commits
Show all changes
874 commits
Select commit
Hold shift + click to select a range
60104a5
update background colour of list-view and status-bar
Yoochard c93367e
Merge branch 'master' of https://github.com/CS2103JAN2018-W15-B1/main
Yoochard 8594271
change filled text to reduce warning when ui starts
Yoochard 0ee7969
make add photo commmand undoable
crizyli 8afdb30
junit tests
crizyli 9de2037
junit tests
crizyli ca23c65
fix style error
crizyli abf8fc0
fix test error
crizyli 432ab26
Merge pull request #201 from crizyli/make_undo
crizyli d1cbda9
update collate
crizyli 369d300
update collate file
crizyli 090ae85
Merge pull request #202 from crizyli/update_collate
crizyli 797cd92
Merge branch 'master' of https://github.com/CS2103JAN2018-W15-B1/main
Yoochard 5b57aac
Merge pull request #13 from CS2103JAN2018-W15-B1/master
IzHoBX 3deb85b
update suggested commands usage
Yoochard 1c1a7c2
fix the error that a black box will appear in Calendar when ET is locked
Yoochard 5643a79
fix the error of black box on the left
Yoochard 8e489ff
adjust colour
Yoochard 4292c2e
Review: Add system test
emer7 f6b3861
Edit: Fix system test
emer7 7d039d9
Remove notification cards for deleted events
IzHoBX 561a27b
Merge pull request #203 from Yoochard/master
Yoochard d1105e5
Change notification cards index to 1-based
IzHoBX 1a5ee35
Person.java: Format toString for better look
emer7 7c046f1
update collate .md file
Yoochard 13090b4
Merge pull request #205 from Yoochard/master
Yoochard 061547c
Disable PersonEditedEvent
emer7 bad8f20
Merge pull request #206 from emer7/master
emer7 58f1b93
implement export command
crizyli 4da7b29
Changes implementation of delete notification
IzHoBX c89bcdb
fix style error
crizyli f89d82c
fix test error
crizyli fab3a1d
Merge branch 'master' into master
crizyli 873d336
Does not show notification for deleted person
IzHoBX d9c3694
add portfolio
crizyli 8c9e27f
Add dismiss command
IzHoBX ff82a9e
Changes implementation of dismiss command
IzHoBX 276589c
Update index of notification cards at the lower of the list when dism…
IzHoBX 6ecf6f5
Add email command
IzHoBX 8ede552
Add emailall command
IzHoBX 27cc973
UGDG update
Yoochard f8e27ea
create PPP
Yoochard 0911e0a
update responsibilities
Yoochard aa7ffd7
UserGuide.adoc: Polish
emer7 0c85a57
emer7: Add personal portfolio page
emer7 f208fa7
Collate emer7
emer7 31f9b03
Fix EOF new line
emer7 120c688
update menu-bar label font
Yoochard 2942894
Merge pull request #210 from Yoochard/master
Yoochard b2c9595
partial fix: remove notification cards when deleting person
IzHoBX 589a476
Merge pull request #208 from crizyli/master
crizyli 325f463
Merge branch 'master' into master
emer7 43d08a0
Merge branch 'master' into ppp
crizyli 3e2ac37
Merge pull request #211 from emer7/master
emer7 b69ddfd
Re-collate
emer7 8ab4d2d
Merge branch 'master' into ppp
crizyli adf02fc
Merge pull request #213 from emer7/master
emer7 953d54c
Fix collated code
emer7 7f91b29
Merge pull request #214 from emer7/master
emer7 86c79ca
Revert "PPP and other documentations"
Yoochard e46d701
Merge pull request #215 from CS2103JAN2018-W15-B1/revert-210-master
Yoochard 79fdd9b
Revert "Revert "PPP and other documentations""
Yoochard 76537a3
Merge pull request #216 from CS2103JAN2018-W15-B1/revert-215-revert-2…
Yoochard e540751
fix style issues and tests
IzHoBX e17b33f
remove irrelevant tests
IzHoBX 6f7d4b2
Merge branch 'master' into master
IzHoBX 3f43980
Merge pull request #212 from crizyli/ppp
crizyli bed7108
Merge pull request #217 from IzHoBX/master
IzHoBX afdb68e
Checkstyle: Fix violations
emer7 bce5c04
Merge pull request #220 from emer7/master
emer7 4ed3b08
revert commit to fix travis error
Yoochard 021aa07
fix photo issue whit jar version
crizyli b47e558
Delete DeveloperGuide.pdf
crizyli 7700592
Delete UserGuide.pdf
crizyli fcb8a09
Update UserGuide.html
crizyli cbd9949
Update build.gradle
crizyli c71349c
Update build.gradle
crizyli 86e3d32
Update build.gradle
crizyli 9079f80
fix edit command system test
crizyli fd00cf9
Merge branch 'master' of https://github.com/crizyli/main
crizyli 47b6226
Merge branch 'master' of https://github.com/CS2103JAN2018-W15-B1/main
Yoochard 2fd9f01
Delete DeveloperGuide.pdf
crizyli 5e805cc
Delete UserGuide.pdf
crizyli f8bcb5e
Update UserGuide.html
crizyli 12f192c
fix bug with travis long pause
crizyli 7385bc4
Merge branch 'master' into master
crizyli 4f9eec4
Merge pull request #222 from crizyli/master
crizyli 8410393
Merge branch 'master' of https://github.com/CS2103JAN2018-W15-B1/main
Yoochard 9c60edc
Update AboutUs.adoc
crizyli 46e90a6
Delete UI.PNG
crizyli 8a6826e
Delete Ui.png
crizyli e694eb4
Add files via upload
crizyli 2a1292b
Update README.adoc
crizyli 6694af3
update command box
crizyli 0eff2fa
Delete UI.png
crizyli 2851c5c
Add files via upload
crizyli e6a2aca
Update README.adoc
crizyli 9e0abd6
update user guide
crizyli f4cc179
Merge remote-tracking branch 'upstream/master'
crizyli ecfd942
SampleDataUtil.java: Update sample employees
emer7 e9019f5
logic.commands: Standardize MESSAGE_USAGE
emer7 e82064c
CommandBox.java: Add missing commands
emer7 feaa572
Remove reference to address book and its terminology
emer7 7f8182f
Merge pull request #223 from emer7/master
emer7 59f49ce
fix tiny issue with deleting calendar
crizyli 776c7e0
Merge branch 'master' of https://github.com/CS2103JAN2018-W15-B1/main
Yoochard 4625ac6
Merge branch 'master' into master
crizyli ec5e631
Merge pull request #224 from crizyli/master
crizyli baddf4f
PersonEditedEvent: Re-implement
emer7 e565daf
Fix systemtests
emer7 2d4184f
Merge pull request #225 from emer7/master
emer7 c894a24
Merge pull request #14 from CS2103JAN2018-W15-B1/master
IzHoBX 1db706f
Fix comments and small details
emer7 bcc9723
Fix bug where unlock command returns invalid command error if redunda…
IzHoBX f871939
Remove notification cards associated with a person when deleting that…
IzHoBX 65d0ddd
Makes the error message of dismiss command more consistent
IzHoBX 755e526
Edit collate
emer7 d5f5754
Collate
emer7 7b47d8f
Merge pull request #226 from emer7/master
emer7 bda8f95
Edit style for notification card
IzHoBX 5106833
Bug fixes for test
IzHoBX c861cbb
Remove outdated test
IzHoBX 1fe9fb3
Merge pull request #221 from Yoochard/master
Yoochard c1dd19f
fix style issues
IzHoBX 645a3da
Update AboutUs.adoc
Yoochard 43c7b9a
Merge pull request #227 from IzHoBX/master
IzHoBX 71b9810
Collates code for v1.5rc
IzHoBX f4e765a
Collate EmailAllCommand
IzHoBX 036986d
Merge branch 'master' of https://github.com/CS2103JAN2018-W15-B1/main
Yoochard aae4a8d
update DG
crizyli 8a33726
adjust BrightTheme
Yoochard 977fc35
Modifies Javadoc comments on input validation
IzHoBX c8b6cd5
update DG
crizyli 3d2bae3
update ppp
crizyli a6ce6d0
Merge pull request #228 from crizyli/update_DG
crizyli e9659e4
Adds tests for Notification
IzHoBX d30315a
Adds Notification System Test
IzHoBX 1137c23
fix style issues
IzHoBX 151f1ed
Defensive coding
IzHoBX 21c2b49
Reviews code and defensive coding
IzHoBX f278b3d
Collate tests
IzHoBX 3daf9de
Fixes regression issues
IzHoBX 28a450b
Merge branch 'master' into master
IzHoBX 2fc1a59
Merge pull request #229 from IzHoBX/master
IzHoBX eea43cb
tidy up code
crizyli 4c713a2
Employees Tracker starts with empty data instead of sample data
emer7 a15ef0b
test travis with authentication
crizyli 24ffa4c
fix style error
crizyli 274ebd2
fix style error
crizyli 5ed8666
SampleDataTest.java: Fix test
emer7 d5b3ce0
Checkstyle: Fix violation
emer7 63bbb48
configure travis.yml
crizyli 34618ef
configure travis.yml
crizyli 632e550
Revert "configure travis.yml"
crizyli 4147352
Revert "configure travis.yml"
crizyli ee2d737
comment test
crizyli f09d89e
test travis
crizyli 2a15628
test travis
crizyli 0f2cbb1
style error
crizyli ee63a59
update tests
crizyli 5580426
fix style error, update collate
crizyli ad3386e
Update UserGuide.adoc
crizyli d64867c
update message
crizyli 4e1d6d1
re collate
crizyli 6083647
recollate
crizyli 17324a3
Merge pull request #230 from emer7/master
emer7 c49ef5a
Merge branch 'master' into master
crizyli 2581239
fix typo
crizyli 5dfeaff
Merge branch 'master' of https://github.com/crizyli/main
crizyli aacdcd6
Merge pull request #231 from crizyli/master
crizyli 0256d2b
Merge branch 'master' of https://github.com/CS2103JAN2018-W15-B1/main
Yoochard 5b5bced
Update user guide
IzHoBX 7c53c53
Merge branch 'master' of https://github.com/IzHoBX/main
IzHoBX 2f34de4
Merge branch 'master' into master
IzHoBX d63b988
Update Figures reference numbering
IzHoBX 58cd711
Merge pull request #233 from IzHoBX/master
IzHoBX 45ac7d4
Merge pull request #15 from CS2103JAN2018-W15-B1/master
IzHoBX 98576c3
ReviewCommandSystemTest: Add tests
emer7 cf2f85b
Remove unused code
emer7 0bcf7ef
Fix typo
emer7 1c32880
ReviewCommandSystemTest.java: Add tests
emer7 11190e4
Fix typo
emer7 41e1c7c
Checkstyle: Fix violation
emer7 1b5890f
update AboutUs and ppp
Yoochard 1b2d13e
update UGDG, seq diagram
Yoochard 8ccc359
adjust codes.
Yoochard 70eab80
Fix mistake
emer7 25bee1a
update collate
Yoochard 02120df
Merge pull request #232 from emer7/master
emer7 a2caec4
update DG instruction for manual testing
crizyli 514e159
Merge remote-tracking branch 'upstream/master'
crizyli 0cdc04a
Merge pull request #235 from crizyli/master
crizyli 083196c
Merge pull request #234 from Yoochard/master
Yoochard 486f20b
Update UserGuide.adoc
emer7 e565415
Fix merge conflict
emer7 f3f8d7a
Merge pull request #236 from emer7/master
emer7 1a556ff
update Ui.png and UiLabelled.png to be coherent with latest version
Yoochard 699b5b1
tell user explicitly that undo a "delete" command may cause some error
Yoochard f3ca1dd
Merge branch 'master' into master
Yoochard 3411d00
Merge pull request #237 from Yoochard/master
Yoochard 90795b5
Merge remote-tracking branch 'upstream/master'
crizyli b103d55
UG fix wrong figure indexing
crizyli 0f51f41
DG update
Yoochard 45454d2
Merge branch 'master' of https://github.com/CS2103JAN2018-W15-B1/main
Yoochard 2d4e2ee
Merge pull request #238 from crizyli/master
crizyli 6140efe
Update Developer Guide
IzHoBX 3b55bfa
Solves code snippet breaking issue
IzHoBX 033305b
Formatting issues
IzHoBX c26de8f
Merge pull request #239 from Yoochard/master
Yoochard 495cf1e
Merge pull request #240 from IzHoBX/master
IzHoBX 60b552b
Update DeveloperGuide.adoc
crizyli 2b72036
Update UGDG
IzHoBX 67b6de5
Include Portfolio
IzHoBX fd8eb11
Merge branch 'master' into master
IzHoBX b15bd27
Update AboutUs
IzHoBX 51a5f9b
Merge pull request #241 from IzHoBX/master
IzHoBX bf71a3a
Update AboutUs
IzHoBX 063645a
Save changes
IzHoBX 7367e2c
Update AboutUs
IzHoBX 24c7865
Delete JohnDoe.adoc
IzHoBX 23cfd6e
Documentation
emer7 c097ff3
Collate
emer7 3becf49
Merge pull request #243 from emer7/master
emer7 7396244
Merge pull request #242 from IzHoBX/master
IzHoBX 2ff0d51
UserGuide: Update image
emer7 9b8ef89
Merge pull request #244 from emer7/master
emer7 f158222
Re-collate
emer7 356413a
Merge pull request #245 from emer7/master
emer7 8a5e12d
AboutUs.adoc and PPP
emer7 ab00fe4
Merge pull request #246 from emer7/master
emer7 38dc3fb
Fix capitalization
emer7 f059d8b
Merge pull request #247 from emer7/master
emer7 11e1c6a
Update README.adoc
emer7 cde4eb3
Update Class Diagram
emer7 c278c4f
Merge pull request #248 from emer7/master
emer7 a3145a5
Final documentation update
IzHoBX fb13267
Merge branch 'master' into master
IzHoBX d7e349f
Merge pull request #249 from IzHoBX/master
IzHoBX b2c3cd9
Adds IzHoBX.html
IzHoBX 2551f98
Deletes IzHoBX.html
IzHoBX f5761ad
Update Yoochard.adoc
Yoochard 2dc1ff5
Update Yoochard.adoc
Yoochard 75c647b
Update Yoochard.adoc
Yoochard d155964
update ppp html version
Yoochard 80e5895
new line added at EOF
Yoochard 5dd1896
Merge pull request #250 from Yoochard/master
Yoochard 3888d7c
Add files via upload
crizyli b361fdd
Delete crizyli.html
crizyli d5db7a7
Add files via upload
crizyli 365a981
Update crizyli.html
crizyli File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Oops, something went wrong.
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.
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.
I guess your project is not Address Book anymore :p, please update the Address Book mentions from all instances. You might also need to update or remove stuff that relates only to address-book and not to your project.