ManuelDeLeon / viewmodel

MVVM for Meteor
https://viewmodel.org
MIT License
205 stars 23 forks source link

[SOLVED] Meteor 1.4 Viewmodel events don't work #274

Closed scott-cornwell closed 8 years ago

scott-cornwell commented 8 years ago

Events don't seem to work with Meteor 1.4

ManuelDeLeon commented 8 years ago

Make a repro. Events work just fine.

fvpDev commented 8 years ago

Don't forget the change in vm where events is not a definable property anymore because it's better to use bindings (especially since there is the repeat binding now). See here: https://viewmodel.org/docs/bindings#events

Something I would forget sometimes, so I hope this helps.

ManuelDeLeon commented 8 years ago

@fvpDev repeat is React only :)

fvpDev commented 8 years ago

Oh, whoops. Now I bet ur really annoyed lol sorry about that. Thanks for the clarification. All the more reason to use React 😄

scott-cornwell commented 8 years ago

@ManuelDeLeon Sorry I just solved my issue. I was starting a new Meteor 1.4 project and doing meteor add manuel:viewmodel gave me viewmodel version 1.9.12 for some reason! I checked that the latest is 6 and tried to update but no luck. Seems to be a Meteor bug. Had to do meteor add manuel:viewmodel@6.0.0 and it updated and now events are working!

I will say there is still some confusion on my part. I guess the version of viewmodel I was using with Meteor 1.3 was 4.1.9. It seems as though {{b "value: message"}} syntax no longer works. I was able to get it working with b="value: message" syntax. Is there somewhere that these changes are mentioned? I still see a lot of examples on your site using the presumably old method, and also the events object on viewmodel (which seems to still work!?). Actually I have that working with Meteor 1.4 but not the event binding method.

Anyhow, thanks for all your work, viewmodel is great! I think I have it working now even though there is some guessing and confusion still on my part.

ManuelDeLeon commented 8 years ago

b=" is for React {{b " is for Blaze

On Sep 27, 2016 5:38 PM, "scott-cornwell" notifications@github.com wrote:

Closed #274 https://github.com/ManuelDeLeon/viewmodel/issues/274.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ManuelDeLeon/viewmodel/issues/274#event-804707317, or mute the thread https://github.com/notifications/unsubscribe-auth/AED31vYC4ur-GbDDYYi37ce8NqTe0IFOks5quaj7gaJpZM4KINcE .