search

Querijn / BottyMcBotface

The Riot Games API Bot
39 stars 29 forks source link

Allow multiple lookups with !lookup #190

Open stelar7 opened 5 years ago

stelar7 commented 5 years ago

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

Querijn commented 5 years ago

Anything separated by whitespace or a comma?

stelar7 commented 5 years ago

both whitespace and commas are used in items / perks iirc

Querijn commented 5 years ago

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

stelar7 commented 5 years ago

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

Querijn commented 5 years ago

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