search

GeyserMC / GeyserDiscordBot

A bot for the GeyserMC Discord server
https://geysermc.org
MIT License
22 stars 42 forks source link

Add OVH tag, remove android.tag, adjust other tags slightly #295

Closed onebeastchris closed 1 year ago

onebeastchris commented 1 year ago

Since https://github.com/GeyserMC/GeyserWiki/commit/d99084701f1521dcf89c2ef242af0737ff990b96, providers such as OVH and OCI are no longer on the providers list - hence, a new tag was needed (!!ovh/!!oci). Additionally, I removed the mcsrvstatus tag in favor of the updated connectiontest command - it has a few more benefits, such as checking for the OVH firewall issue, checking for config mismatches, and telling the user whether they are checking their own server :p

Kas-tle commented 1 year ago

https://github.com/GeyserMC/GeyserDiscordBot/actions/runs/6099206156/job/16550806978?pr=295#step:4:64

onebeastchris commented 1 year ago

https://github.com/GeyserMC/GeyserDiscordBot/blob/master/src/test/java/org/geysermc/discordbot/commands/search/ProvidersTest.java#L50 It looks like the removal of the Google Cloud provider from the wiki's providers list caused the test to fail :p I'll fix it once I'm back home

onebeastchris commented 1 year ago

yay fixed it, changed the check from Google cloud -> Cloud Nord