search

eclair4151 / MC-Status-Widget-for-Minecraft

A powerful tool for checking the status of your Minecraft Server. Available on iOS, WatchOS, and macOS
https://itunes.apple.com/us/app/server-widget-for-minecraft/id1408215245
47 stars 4 forks source link

App not displaying correctly for bedrock servers #39

Closed benshan36 closed 2 months ago

benshan36 commented 2 months ago

The app was working fine for me for a while, and then a few months ago it started displaying the info incorrectly. It is showing version as the number of players, MOTD shows the protocol and game mode, and number of players just says 20/0 constantly. Please fix this IMG_3543

eclair4151 commented 2 months ago

Thanks ofr the report, I think I've fixed this in my latest update, do you mind emailing me the server details, so I can confirm the fix on my end? tomerjshemesh@gmail.com

Thanks!

benshan36 commented 2 months ago

Yes I’ll send you the info. Thank you so much!

eclair4151 commented 2 months ago

This was due to parsing bug, I incorrectly assumed stringInput.split(separator: ";")

would preserve blank strings where double ;; appeared, but I had to instead specify stringInput.split(separator: ";", omittingEmptySubsequences: false)

https://github.com/eclair4151/MC-Status-Widget-for-Minecraft/blob/92f34dcbe6fced83f2eb25ea063b5d25aa64907f/MinecraftServerStatus/Helpers/StatusChecker.swift#L299

eclair4151 commented 1 month ago

This is now complete. You can test the new 2.0 version of the app here on testflight before release:

WARNING!! There is currently a bug that means you will have to delete and re-add your widgets if you try out the new beta. This should be resolved before release but I didnt want to hold up testing.

https://testflight.apple.com/join/UqHNAYJn

benshan36 commented 1 month ago

Yes, the bug was still prevalent for me as well but only a minor inconvenience. The app rebuild looks great! And my server displays its information properly again. Thank you so much

On Fri, Oct 18, 2024 at 18:21 Tomer Shemesh @.***> wrote:

This is now complete. You can test the new 2.0 version of the app here on testflight before release:

WARNING!! There is currently a bug that means you will have to delete and re-add your widgets if you try out the new beta. This should be resolved before release but I didnt want to hold up testing.

https://testflight.apple.com/join/UqHNAYJn

— Reply to this email directly, view it on GitHub https://github.com/eclair4151/MC-Status-Widget-for-Minecraft/issues/39#issuecomment-2423361367, or unsubscribe https://github.com/notifications/unsubscribe-auth/BK2PJOP4PTVYGDIBLKA52P3Z4GJV3AVCNFSM6AAAAABNGUSX7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRTGM3DCMZWG4 . You are receiving this because you authored the thread.Message ID: @.*** com>