search

skifaster / meteor-app-update

MIT License
11 stars 5 forks source link

If a notification is currently being displayed and another code push comes through, the notification disappears #1

Closed keviniamburg closed 9 years ago

keviniamburg commented 9 years ago

Not sure if this is expected behavior or not, but wanted to bring it up. Cheers.

skifaster commented 9 years ago

Ah. Thanks for the heads up. Will take a look. This is not expected behavior.

skifaster commented 9 years ago

@slickdev i was unable to reproduce this issue consistently. I did see the issue once or twice during my testing, but not enough for a proper repo. Do you have a repo I can try to recreate it more consistently?

Here are my current settings for the package. I have this in my lib folder for testing: 

if(Meteor.isClient) {
    AppUpdate.reload.settings.environments.development = true;

    AppUpdate.reload.preventReload();
}
keviniamburg commented 9 years ago

I ended up removing the plugin in the end, since I couldn't get it to work reliably enough. I might give it another try on another project, will keep you posted.

skifaster commented 9 years ago

Ok. Thanks for the heads up. I'll close this issue for now.