Testing confirm about ng-snotify HOT 8 CLOSED

I saw this module today and I love it, it has more option and functionality that I like. lemme go through it and give you my feedback where you could improve on.

from ng-snotify.

I'am not sure that you can do this at all.
But looks like a nice idea to work about.
I'll try to implement it this weekend

from ng-snotify.

@artemsky this is nice project, could you reduce the amount of code and how to print a message without the tittle or even pass your own massage like ng2-flashmessage.

from ng-snotify.

@gbrigens I'll try to do something with it)
Could you create new issue with your suggestion, and add more info (link) about ng2-flashmessage

from ng-snotify.

@maxxt Sorry for your waiting. Hand no time, to review.
I found that you can call that action.

When you calling any type of toast it returns self id
Then you can get current toast object with snotifyService.get(id) that represent the following structure

{
   "id":1055795966222,
   "title":"Snotify title!",
   "body":"Lorem ipsum dolor sit amet!",
   "config":{
      "showProgressBar":true,
      "timeout":3000,
      "closeOnClick":false,
      "pauseOnHover":true,
      "buttons":[
         {
            "action": function()
            "text":"Yes",
            "bold":false
         },
         {
            "action": function()
            "text":"No",
            "bold":true
         }
      ],
      "placeholder":"Enter answer here...",
      "bodyMaxLength":80,
      "titleMaxLength":15,
      "backdrop":-1,
      "type":6
   }
}

So you can call you action by accessing this object.

const id = snotifyService.confirm("title", "body", {
      buttons: [
        {text: 'Yes', action: () => doSomeJob(), bold: false},
        {text: 'No', action: () => doAnotherJob(), bold: true},
      ]
    });

snotifyService.get(id).config.buttons[0].action()

For testing, its fine.
But its not recommended to call it pragmatically in production for now. I have an issue, where toast is not hiding, if you calling snotifyService.remove(id) inside action;
Need to fix it.

from ng-snotify.

@artemsky thanks for the reply.

Inside the action I am not passing an ID as a param to remove() and it works just fine.

from ng-snotify.

@maxxt Without id it removes all notifications)

from ng-snotify.

@artemsky Nope, in my case it doesn't.

from ng-snotify.