search

mikeshardmind / SinbadCogs

A set of community resources
Apache License 2.0
50 stars 47 forks source link

Clarify help for bulkrolebind syntax #214

Closed McGarn closed 4 years ago

McGarn commented 4 years ago

I've included two + characters hoping they're properly escaped by the string block

mikeshardmind commented 4 years ago

Thanks for taking an interest in making this a little clearer for other people.

Unfortunately, I don't think these specific instructions are a good way to recommend people interact with this command.

I'm aware of a large enough group of people using my cogs primarily from mobile that I've added a couple of specific helpers for mobile-specific behavior in a few places, and while some mobile keyboards handle shift + enter identically to desktop, not all of them do.

While a new line makes the message easier to visually see the pairs, a series of spaces does also still work here.

The cog can also parse the roles without mentioning them (either by ID or by an exact name match)

If you have a recommendation for a help string that can make all of that clear enough, I'd be happy to merge it, but I'd like to avoid adding instructions that may not work for a significant portion of users.

DiscordLiz commented 4 years ago

The help has been made clearer with a different approach.