search

PredaaA / predacogs

Cogs for Red Discordbot
MIT License
30 stars 25 forks source link

[MarTools] Add new voice regions #70

Closed Ryan5453 closed 2 years ago

Ryan5453 commented 2 years ago

Fixes: https://sentry.io/share/issue/726cfb4a00f14657a6bd6eb4e080096e/

Ryan5453 commented 2 years ago

Do you think it would be a good idea to try/except the dict and display something like 'Other' if a KeyError was raised in case discord randomly adds more VC regions?

PredaaA commented 2 years ago

Thanks for the PR! While we're on adding voice regions, there are some others, here's all of them: atlanta, newark, santiago, santa-clara, st-pete, buenos-aires, seattle, stockholm

Do you think it would be a good idea to try/except the dict and display something like 'Other' if a KeyError was raised in case discord randomly adds more VC regions?

Yes, something like flag=regions_dict.get(region_name, region_name), would works.