Possible typo in readme might cause a glitch
guy-roberts opened this issue · 2 comments
guy-roberts commented
Alright Kris,
Been trying out wisper-activerecord for indexing ElasticSearch when records change. In the readme.md it says (once);
meeting.subscribe(Auditor.new)
Auditor is a Singleton, should this be
meeting.subscribe(Auditor.instance)
Wonder if that's why people were getting that message about a class being unloaded from the module tree ?
krisleech commented
Hey! You are right it should be instance
not new
. Have a biscuit 😉
I'll leave this issue open so I remember to fix it.
I think I did new
for the initial example because some people might not be familiar with instance
. Maybe I should either change the first example to something else or change Auditor
to not be a Singelton...
guy-roberts commented
I had a Custard Cream, thanks.