Safwan Rahman

Results 78 comments of Safwan Rahman

I think `_send_notifications` is an internal API. therefore from the user end, it should not be used.

Yap. Comment should be added what type its expecting. But it will be not a good idea to use it from outside.

> shouldn't the json.dumps(payload) be in the _send_notifications? I think it should not be there. Can you please explain why it should be there?

@jamaalscarlett Actually, according to specification, payload only allows text. we make the dictionary to json so we can send the text to browser then parse the json from the text....

@timo-piirmann Good observation! I have never thought about it! Thanks a lot. This need to be fixed with priority

Interesting. Need to look at it. @saadmk11 Will you have some time to look?

@pausanchezv Which Elasticsearch version?

@herrnici Can you explain your usecase?