MDL-57139 messages/notifications: ensure promise best practices

MDL-57139 messages/notifications: ensure promise best practices

Previously there were a few issues with the code

* We were capturing a promise without then then .then() that came after
  it, so our promise wouldn't always be completely resolved by the time
  then next piece of code was operating on it

* We weren't catching all errors with .catch()