search

PaarthShah / matrix-nio-hacs

A HACS Integration to replace the core matrix Home Assistant integration.
14 stars 3 forks source link

fixing image upload #8

Closed neubi4 closed 1 year ago

neubi4 commented 1 year ago

Image Upload is currenty broken because nio client.upload() returns a tuple instead of only the reponse. Docs: https://matrix-nio.readthedocs.io/en/latest/nio.html#nio.AsyncClient.upload

This PR fixes this.

PaarthShah commented 1 year ago

@neubi4 Ahh, dang, that's a good catch! Ty for making this! Though it looks like the tests need to be updated as well; think you can take that on?

neubi4 commented 1 year ago

@PaarthShah mock is updated, tests should pass now.