Skip to content
This repository has been archived by the owner on Mar 13, 2019. It is now read-only.

added room info command #4

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

added room info command #4

wants to merge 9 commits into from

Conversation

szero
Copy link

@szero szero commented May 16, 2017

This uploads info text file to the room. I made this because I see people asking about who is room owner constantly.
Also I think it would be sensible to add owner column to rooms table in phrases2.db since vola config have owner field now. I will try to make that.

requirements.txt Outdated
@@ -1,7 +1,7 @@
isodate>=0.5.1
python-dateutil>=2.4.2
requests>=2.7.0
volapi>=5.0.0
volapi>=5.1.1
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make it 5.2.0 because new feature

@@ -2,7 +2,7 @@

__all__ = ["__version__", "__title__", "__fulltitle__"]

__version__ = "3.0"
__version__ = "3.1"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't mess with version in feature commits

return True
RoomInfoCommand.last_check = time()
info = []
info += "{:>20}: {}".format("Owner", self.conf_key("owner")),
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you want a shorthand of self.room.config just do config = self.room.config and use that. do not add a method

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants