search

thervh70 / ContextProject_RDD

1 stars 0 forks source link

65 event handlers #101

Closed MathiasMeuleman closed 8 years ago

MathiasMeuleman commented 8 years ago

Will close #65

There are now several RawDataTrackers, based on the interfaces created in #64 Every Tracker tracks a single event and logs this to the database by using it's personal DatabaseAdaptable.

TODO: test (we need #67 for this)

Please have a look at it :)

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-3.1%) to 64.277% when pulling 0a8b069fd84c55663c4185f7f9c56077808a142e on 65-event-handlers into 3095e6f52c883fccc1d9f003ce7caad99d772097 on dev.

mpsijm commented 8 years ago

Mathias, as you're being a MAS TA now, just give a :+1: for the things you like and I'll change them for you if you don't have any time :)

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-3.1%) to 64.277% when pulling 7ccde4d6c169109ac3d8610d4070ed11c2639cb8 on 65-event-handlers into 3095e6f52c883fccc1d9f003ce7caad99d772097 on dev.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+15.7%) to 83.096% when pulling 55ae40d77a4ec72f62a06aa14b9a54b7a6d130f0 on 65-event-handlers into 3095e6f52c883fccc1d9f003ce7caad99d772097 on dev.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+15.7%) to 83.096% when pulling 52619db5643f3918d25da2a9754aa347ff770539 on 65-event-handlers into 3095e6f52c883fccc1d9f003ce7caad99d772097 on dev.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+16.3%) to 83.672% when pulling eb4d039da913718f7c00ed10a088c46d1a45360d on 65-event-handlers into 3095e6f52c883fccc1d9f003ce7caad99d772097 on dev.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-3.9%) to 85.954% when pulling ff56a22c50a971a05d974da628b9fff7af3a11fc on 65-event-handlers into 756bfcb75da6db68dc9c5dc9361e24fec41a862b on dev.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-2.2%) to 87.629% when pulling c0bde1ab249bd68266dd5a2f3b7baf196d3de858 on 65-event-handlers into 756bfcb75da6db68dc9c5dc9361e24fec41a862b on dev.

mpsijm commented 8 years ago

Merge approved!

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-2.2%) to 87.629% when pulling c0bde1ab249bd68266dd5a2f3b7baf196d3de858 on 65-event-handlers into 756bfcb75da6db68dc9c5dc9361e24fec41a862b on dev.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-2.2%) to 87.629% when pulling cbf3744c0c8d6030d4467dd9a3443f75e4746d5c on 65-event-handlers into 756bfcb75da6db68dc9c5dc9361e24fec41a862b on dev.

mdingjan commented 8 years ago

I've reviewed your changes. This is ready for merging! :)

Will merge!