-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Conversation
Updated to 12 max members and 60 for token length
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You did not read the checklist! this is not exported from a panel
It's the same, I don't know how to make it otherwise |
I screwed up a bit, hope now its all ok |
yea, no problem. We don't bite. I exported it from the panel so technicly I can merge this but you had a build error I saw. |
In theory it shouldn't be because of my changes since the server hasn't started yet and the original docker I'm already using has always worked with these settings |
Must admit that I didn't look too closely at which values should be allowed there since I don't use them, so thanks for the PR. It seems indeed that the yuzu team has made some changes to their source code in the meantime, making that warning pop up. However, compilation still succeeds as its only a warning (just ran a reinstall and that worked). Maybe there is some way to suppress that warning, although they can of course also be useful for debugging if something goes wrong, so I am not sure if that is something we want to do. |
LGTM! |
Description
Updated to 12 max members and 60 for token length
Checklist for all submissions