BottyMcBotface icon indicating copy to clipboard operation
BottyMcBotface copied to clipboard

Allow multiple lookups with !lookup

Open stelar7 opened this issue 5 years ago • 5 comments

Ie. searching for more than one champion at a time !lookup champion neeko sylas

This should work fine for champions? but would need some special filtering for perks and items as they contain spaces. Another option is to change the syntax, i.e

!lookup champion neeko | sylas

stelar7 avatar Apr 10 '19 08:04 stelar7

Anything separated by whitespace or a comma?

Querijn avatar Apr 10 '19 08:04 Querijn

both whitespace and commas are used in items / perks iirc

stelar7 avatar Apr 10 '19 08:04 stelar7

Add options for quotes, I guess, ie "Twisted Fate". Where are comma's used?

Querijn avatar Apr 10 '19 08:04 Querijn

Seems like i was mistaken about ,, got it confused on BFSword 😅 Id rather prefer a more clear separator like | tho

stelar7 avatar Apr 10 '19 08:04 stelar7

It's uncommon in argument parsing, and possibly very confusing for the end user, but you could say the same of comma's.

Querijn avatar Apr 10 '19 08:04 Querijn