-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Using Meteor toys pro along Redis-oplog #94
Comments
How are you liking that package? Does removing it make Toys work?
…On Thursday, February 16, 2017, David Panart ***@***.***> wrote:
Using redis-oplog in the same project than meteor toys Pro, after I did a meteor
update from 1.4.2.3 to 1.4.2.7 (and almost all the packages in my app), I
now get this when I start my app 👍
Exception while invoking method 'MeteorToys_impersonate_account' { stack: 'TypeError: Cannot read property \'find\' of undefined\n at module.export.exports.default (packages/cultofcoders:redis-oplog/lib/processors/actions/requery.js:14:22)\n at handleRemove (packages/cultofcoders:redis-oplog/lib/processors/limit-sort.js:72:9)\n at module.export.exports.default (packages/cultofcoders:redis-oplog/lib/processors/limit-sort.js:20:13)\n at RedisSubscriber.processSync (packages/cultofcoders:redis-oplog/lib/redis/RedisSubscriber.js:65:9)\n at packages/cultofcoders:redis-oplog/lib/mongo/lib/compensateForLatency.js:19:28\n at Array.forEach (native)\n at packages/cultofcoders:redis-oplog/lib/mongo/lib/compensateForLatency.js:18:25\n at Array.forEach (native)\n at module.export.exports.default (packages/cultofcoders:redis-oplog/lib/mongo/lib/compensateForLatency.js:13:14)\n at packages/cultofcoders:redis-oplog/lib/mongo/lib/dispatchers.js:36:9',
I20170216-13:00:38.373(1)? source: 'method' }
versions file before the update (1.4.2.3) : http://pastebin.com/CR9MxWhC
versions file after the update (1.4.2.7) : http://pastebin.com/8vuKXyHM
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#94>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AB7-g__DU0TV-q33pobgI5vGKjn26D6Vks5rdDwjgaJpZM4MC7rj>
.
|
Yes it works well when I remove redis-oplog. |
I haven't used the package so it's hard for me to identify the issue. I'm
going to look into it soon, but in meanwhile, going to loop in @theodorDiaconu
maybe it's helpful for him to know and/or maybe he could provide some
insight into the problem
…On Thursday, February 16, 2017, David Panart ***@***.***> wrote:
Yes it works well when I remove redis-oplog.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#94 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AB7-gwSGF8e2iE8ot6D0De8P7-zZedkHks5rdFAWgaJpZM4MC7rj>
.
|
Interesting bug. Will look into it. |
@msavin I think I may be on to something, give me a bit of insight of what kind of collections meteor toy uses ? Are they local collections ? |
@Fen747 hey man- there is a new version that had big changes- maybe it makes difference? @theodorDiaconu ah, MT is using this package, which may be problem? |
@msavin most likely yes, the reason was that meteortoys needs to be added after redis-oplog. |
@Fen747 hey man - did @theodorDiaconu recommendation help you get back into using Meteor Toys? |
Using redis-oplog in the same project than meteor toys Pro, after I did a
meteor update
from 1.4.2.3 to 1.4.2.7 (and almost all the packages in my app), I now get this when I start my app :versions
file before the update (1.4.2.3) : http://pastebin.com/CR9MxWhCversions
file after the update (1.4.2.7) : http://pastebin.com/8vuKXyHMThe text was updated successfully, but these errors were encountered: