Toxic-Cogs icon indicating copy to clipboard operation
Toxic-Cogs copied to clipboard

[Reacticket] Permissions issue

Open Zenchaeus opened this issue 4 years ago • 4 comments

Users can't view message history which can cause a lot of problem when they don't have their app open or they leave. image

Zenchaeus avatar Mar 15 '21 05:03 Zenchaeus

i got around this by enabling read message history and disabling view in category settings then letting the bot handle the perms for the channel itself

cry8wolf9 avatar Mar 17 '21 03:03 cry8wolf9

i got around this by enabling read message history and disabling view in category settings then letting the bot handle the perms for the channel itself

Doesn't seem to be working for us, as it isn't copying the category permissions

The only work around would be to completely change all of our categories and channels setups, and then change it so on our server everyone can read message history by default, which isn't happening as it would break functionality of our server

CruxCraft avatar Nov 01 '21 22:11 CruxCraft

https://github.com/NeuroAssassin/Toxic-Cogs/blob/21cbf41173ee0375053e9849f57c04fca8b45580/reacticket/reacticket.py#L377

Adding read_message_history=True to discord.PermissionOverwrite() on line 377 appears to resolve the issue of users not being able to see the bot message in a newly created ticket channel since it gives the "Read Message History" permission to the ticket owner and ticket managers.

fawkinay avatar Nov 02 '21 00:11 fawkinay

https://github.com/NeuroAssassin/Toxic-Cogs/blob/21cbf41173ee0375053e9849f57c04fca8b45580/reacticket/reacticket.py#L377

Adding read_message_history=True to discord.PermissionOverwrite() on line 377 appears to resolve the issue of users not being able to see the bot message in a newly created ticket channel since it gives the "Read Message History" permission to the ticket owner and ticket managers.

Just saw this, but I ended up fixing it this morning... line 383 & I sent a pull request for it...

Edit: I just looked at the line you pointed out, and I don't think that would get each person that requires it... I might be wrong... However Line 383 already sets the default read to false but adding history true let's everyone else who has the permission to view see the history...

CruxCraft avatar Nov 02 '21 21:11 CruxCraft