Bug/#138 initial subscribe value #193

yasserf · 2016-07-27T16:27:44Z

No description provided.

coveralls · 2016-07-27T16:30:14Z

Coverage decreased (-0.02%) to 97.158% when pulling 29325ff on bug/#138-initial-subscribe-value into 45cc40a on master.

yasserf · 2016-07-27T21:49:11Z

This should be merged last as it depended on fixes in other branches

coveralls · 2016-07-27T21:51:36Z

Coverage increased (+0.02%) to 97.2% when pulling 78b35e5 on bug/#138-initial-subscribe-value into 45cc40a on master.

coveralls · 2016-07-27T22:38:45Z

Coverage increased (+0.02%) to 97.202% when pulling f406182 on bug/#138-initial-subscribe-value into 45cc40a on master.

timaschew · 2016-07-28T13:28:48Z

Review was done in #195

…alled via lists

coveralls · 2016-07-28T15:31:40Z

Coverage increased (+0.01%) to 97.228% when pulling f95403b on bug/#138-initial-subscribe-value into 5369db1 on master.

yasserf added the awaiting review label Jul 27, 2016

yasserf force-pushed the bug/#138-initial-subscribe-value branch from 29325ff to 78b35e5 Compare July 27, 2016 21:48

yasserf force-pushed the bug/#138-initial-subscribe-value branch from 78b35e5 to f406182 Compare July 27, 2016 22:36

timaschew mentioned this pull request Jul 28, 2016

Bug/#190 unsubscribe #191

Merged

yasserf and others added 3 commits July 28, 2016 17:27

Using this._normalizeArguments for unsubscribe as its expected when c…

8b73938

…alled via lists

record: Fixed bug with subscription missing initial value.

9678634

Fix #138 adding tests

f95403b

yasserf force-pushed the bug/#138-initial-subscribe-value branch from f406182 to f95403b Compare July 28, 2016 15:28

yasserf merged commit aebca26 into master Jul 28, 2016

yasserf removed the awaiting review label Jul 28, 2016

yasserf deleted the bug/#138-initial-subscribe-value branch July 28, 2016 16:38

Provide feedback