Improve examples, bug fixes

stampit-org / react-stampit

A specialized stampit factory for React

131 stars 1 forks source link

Closed troutowicz closed 9 years ago

troutowicz commented 9 years ago

Relates to issue #9.

deep merge this inside .init closure when using class decorator

Fixes scenario when a mixin is introducing state to a class that declared state in it's constructor
use correct convenience property for stamp decorator test
stampit() instead of wrongly used stampit.init()

cc @ericelliott

ericelliott commented 9 years ago

Can you supply a complete list of the bugs you fixed in this just so I can quickly scan and make sure they're all covered?

troutowicz commented 9 years ago

deep merge this inside .init closure when using class decorator

Fixes scenario when a mixin is introducing state to a class that declared state in it's constructor
use correct convenience property for stamp decorator test
stampit() instead of wrongly used stampit.init()

ericelliott commented 9 years ago

Cool, I'll take a second pass as soon as I can! =)

troutowicz commented 9 years ago

Finished up english descriptions for the rest of the doc examples, merging this.