search

juliancwirko / meteor-s-alert

Simple and fancy notifications for Meteor
https://atmospherejs.com/juliancwirko/s-alert
190 stars 24 forks source link

Not working after update #47

Open rokanost opened 8 years ago

rokanost commented 8 years ago

Hey,

Alerts are not appearing when called after 3.1.2 => 3.1.3 update. Neither on 3.1.4

juliancwirko commented 8 years ago

Could you provide some more informations? Are there any errors in the browser console? Because the demo here: http://s-alert-demo.meteor.com/ uses 3.1.4 and it seems to work.

rokanost commented 8 years ago

No errors. Currently using 3.1.2 and it works fine.

juliancwirko commented 8 years ago

Unfortunately I need some more info because everywhere where i tested it it works.

lscordilis commented 8 years ago

Mine isn't working either.

I'm using on the client

angular                          1.0.0*
angular:angular-material         0.11.4*
urigo:angular-blaze-template     0.3.0

Maybe these interfering with the execution of sAlert, because even when running it in the console nothing appears. I get the id from the return statement but no actual template changes, error messages, or any other indications that something is going wrong.

I'm going to downgrade to 3.1.2 or earlier and see if that changes anything.

juliancwirko commented 8 years ago

could you create a simple reproduction repository?

turbobuilt commented 8 years ago

I'm using angular as well, and it's not working for me either. I tried installing 3.1.2 and that didn't work either. Getting error: template not found for <blaze-template name="sAlert">

Actually apparently blaze template support has been removed from angular meteor. Weird. https://github.com/Urigo/angular-meteor/issues/849

juliancwirko commented 8 years ago

Hi, unfortunatelly, for now if there is no Blaze support in the Angular stack there will be also no possibility to use this package :/ There is a pure React component but I don't have any Angular version.

lscordilis commented 8 years ago

@dtruel @juliancwirko It is possible to use Blaze templates in Angular only with the urigo:angular-blaze-template package

I believe the issue exists in the newer angular-meteor packages. I've locked my version to 1.0.7 for now, because the 1.3+ changes are drastic and require major refactoring.

Even if I were to upgrade anytime soon, it would be unfortunate to lose support for sAlert in angular-meteor because I like it infinitely more than angular-material's alert template.