Skip to content

Commit

Permalink
Set adminonly for resethistory and removed permission
Browse files Browse the repository at this point in the history
  • Loading branch information
@Yepoleb
Yepoleb committed Jun 11, 2014
1 parent 44f535b commit ba2b2e9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion plugins/history.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ def chat_tracker(paraml, input=None, db=None, conn=None):
track_history(input, message_time, conn)


@hook.command(autohelp=False, permissions=["botcontrol"])
@hook.command(autohelp=False, adminonly=True)
def resethistory(inp, input=None, conn=None):
"""resethistory - Resets chat history for the current channel"""
try:
Expand Down

3 comments on commit ba2b2e9

@daboross
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't really see the reason behind this, adminonly is only maintained for legacy support and just translates directly to permissions.

@Yepoleb
Copy link
Author

Choose a reason for hiding this comment

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

I thought it's an easier version of permissions, reverted it in a327dc4

@daboross
Copy link
Contributor

Choose a reason for hiding this comment

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

It is easier I guess but it doesn't allow as fine grained control for groups

Please sign in to comment.