96 implement watch PR and stop watch PR event bindings

mpsijm commented 8 years ago

Will close #96

Because Youri is gone for the weekend, I finished his issue. Main changes:

start/stop watch PR is now being logged
URLHandler.isPullRequestUrl now returns a boolean, as one would expect. The old behaviour (returning an array) has been moved to URLHandler.getPullRequestUrlFields.
Refactored MainController as a result of methods becoming too large because of the extra post methods on multiple places.

Have fun reviewing! :)

coveralls commented 8 years ago

Coverage decreased (-0.7%) to 93.137% when pulling 858d07d2ba19227acb63bc4f67f5cf578cae9c11 on 96_-_Implement_watch_PR_and_stop_watch_PR_event_bindingsImplement_watch_PR_and_stop_watch_PR_event_bindings into 66344d3a6a85290e02fa75932b4711454e5b4b3d on dev.

coveralls commented 8 years ago

Coverage decreased (-0.3%) to 93.557% when pulling 42f2a237903ce9faf1b611c32919fbf29c5a9383 on 96_-_Implement_watch_PR_and_stop_watch_PR_event_bindingsImplement_watch_PR_and_stop_watch_PR_event_bindings into 66344d3a6a85290e02fa75932b4711454e5b4b3d on dev.

coveralls commented 8 years ago

Coverage decreased (-0.3%) to 93.539% when pulling e7bba69203748e0b715db80ccce059fe76ebc65f on 96_-_Implement_watch_PR_and_stop_watch_PR_event_bindingsImplement_watch_PR_and_stop_watch_PR_event_bindings into 66344d3a6a85290e02fa75932b4711454e5b4b3d on dev.

coveralls commented 8 years ago

Coverage decreased (-0.3%) to 93.557% when pulling 30ea659dd675b30e45997d823be4fc82f3e53118 on 96_-_Implement_watch_PR_and_stop_watch_PR_event_bindingsImplement_watch_PR_and_stop_watch_PR_event_bindings into 66344d3a6a85290e02fa75932b4711454e5b4b3d on dev.

coveralls commented 8 years ago

Coverage decreased (-0.3%) to 93.557% when pulling 3f5a481b4989a5e1022631c89870b7a739f62cd9 on 96_-_Implement_watch_PR_and_stop_watch_PR_event_bindingsImplement_watch_PR_and_stop_watch_PR_event_bindings into 66344d3a6a85290e02fa75932b4711454e5b4b3d on dev.

coveralls commented 8 years ago

Coverage decreased (-0.3%) to 93.557% when pulling 3f5a481b4989a5e1022631c89870b7a739f62cd9 on 96_-_Implement_watch_PR_and_stop_watch_PR_event_bindingsImplement_watch_PR_and_stop_watch_PR_event_bindings into 66344d3a6a85290e02fa75932b4711454e5b4b3d on dev.

MathiasMeuleman commented 8 years ago

Yes, nice :) extra functionality is always nice to have. Will merge

thervh70 / ContextProject_RDD

96 implement watch PR and stop watch PR event bindings #144