Although the call of get_room_id (and set_room_topic too) to acquire the id of a group instead of a channel is misleading, as get_room_history is a call on channels, but would be a breaking change otherwise.
Maybe this could be a cleanup for further releases?
Or add an alias for them like get_private_room_id, set_private_room_topic, get_public_room_history and deprecate the others.

Hey @pipozzz can you have a look? Closed most of your open issues and made a few enhancements.

I'm a little busy right now, but definitely I'll review it and let you know. Thanks for contribution.

Also we need to resolve failed build before merging, btw Thanks for your contribution again ! 👍

Hello, @soundstorm!
Are you planning to fix the failed build?
As much as I know, the message from Travis CI not so crucial:

'rocketchat.calls.groups.upload_remote_file.UploadRemoteFile' imported but unused

I'm very new to programming, but interested in the project, thank you!

It seems that code is OK just test coverage dropped below 85%.

This is still in the refresh branch after two years, will there be a release with those changes anytime?

Hi @soundstorm, I was very busy, but Yes I'll review it and merge it if it's fine. There was failed some test last time I took a look. So let me check it again and merge it if it's fine. Thanks