* Minor changes to the help boxes of /ds and /ms
Removed parameters that do nothing (instruct, mimic), changed ssb examples (they were obsolete), added a couple more things. Would love to make the wall of mods you can use with both ds and ms smaller, since a lot of those don't actually change their behaviour, like randomroulette, but I'm not sure how to approach it. A hardcoded list of mods that change things is an option.
* Added "restricted" as a parameter
For a long time, connecting to non-localhost servers didn't work without
an HTTPS cert. But it's now once again possible. All you need is an
IP with an open port, and a dream.
Syntax is a little different than before. To go to `1.2.3.4` at port `5000`
- before: `http://1.2.3.4-5000.psim.us`
- after: `http://1-2-3-4--5000.insecure.psim.us`
Supporting multiple dots in the URL is too complicated, unfortunately.
The other way still works: `http://1.2.3.4:5000`
And yes, I would 100% recommend that one.
* Improve host/gameban
- Allow for offline namehostban and namegameban.
- Notify users when they get host and gamebanned.
- Fix hostbans, which did not work at all.
- Fix host and gamebanned users from being able to subhost
* Change aliases and remove gameban unhostbanning.
Today, a bug was found where using revealas on a long role crashed PS!
I alleviated this by adding a maximum role length (instead of having it be uncapped, as currently people could spam PS using unlimited length roles) and moving the 'infinite loop' check to not be triggered by long roles.
Co-authored-by: Meijer,L. (Lucas) <l.meijer6@students.uu.nl>
Nitpick; the button to check out a text ticket shows as "Claim" when someone else is already looking at the ticket or it is already closed, which doesn't really make sense.
https://www.smogon.com/forums/threads/clean-up-update-error-message-for-faq.3764560/
!faq fails no longer send the failed message message in chat, this is very helpful for room auth who are the users of broadcast commands
For whatever reason '!faq all' fails used a replybox? I can't think of a single non dev command that uses visible replyboxes in chat for fails, so changed this for an errorReply.
I cleaned up the wording of the /help faq, and I added some topics that were not there previously.
I figured PR #11105 would have some unintended side effect, but this
one isn't bad at all. There's no longer an easy way to tell whether
requests are move requests or not, but it's easy enough to instead
track the current turn.
* !rfaq fail over to !faq
Approved Suggestion: https://www.smogon.com/forums/threads/have-rfaq-fail-over-to-faq-if-a-match-is-not-found.3687458/
Code calls for RFAQ topics if that fails it checks the FAQ topic list. Sounds straightforward until you realize the faq broadcast fail command is archaic and for some reason the failed FAQ broadcast message still goes through on top of the actual faq error return message, along with the faq help, it's very messy I might just fix that next. This is bypassed by having the run broadcast call AFTER the topic is read as valid, so none of this nonsense can happen. I really thought this would be easy, turns out, not really (atleast for me).
Fix /removedaily, adding a new "all" option to work like it's
documented to work, and fix the documentation.
Also remove support for legacy image format. A lot of functions were
async that didn't need to be.
Also remove an entirely unused sorting pass mistakenly introduced in
11ab3d83f8
(The queue was sorted, then immediately ignored to iterate it by index,
which is for the best, because showing the queue in any order other
than queue order is not useful.)
Replace outdated (like, since XY outdated) links with current links.
I don't *love* dumping users at the Smogon Metagames forum, but there is no where else I am aware of that has all of our formats listed out.
Also updated the description to explain how /tier works and explain what type of information it contains.
https://www.smogon.com/forums/threads/clarification-on-how-long-a-ban-lasts.3761482/
2 day bans do not say the amount of time which can be confusing to new users as they think a room ban is permanent. This commit does so without influencing week room ban text logs which already do note the 1 week time limit. This is an approved suggestion on the forums as well.
