Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Change "That nation is not public." #1524

Closed
Basically-Andrew opened this issue May 31, 2022 · 8 comments
Closed

[Bug] Change "That nation is not public." #1524

Basically-Andrew opened this issue May 31, 2022 · 8 comments
Labels
Issue type: Bug Something isn't working

Comments

@Basically-Andrew
Copy link
Contributor

Server

Aurora

Describe the bug

Players are shown the message "That nation is not public." when trying to teleport to a nation that isn't allied.

To Reproduce

/n spawn unAlliedNation

Expected behavior

Change Towny's message to a more appropriate message that doesn't confuse players.

Screenshots

No response

Additional context

No response

@Basically-Andrew Basically-Andrew added the Issue type: Bug Something isn't working label May 31, 2022
@ghost
Copy link

ghost commented May 31, 2022

This isn't an EarthMC "Bug", go report this issue at https://github.com/TownyAdvanced/Towny or make the changes yourself there with a Pull Request.

@Basically-Andrew
Copy link
Contributor Author

This doesn't seem to be a Towny bug as this didn't work on Nova. Appears to be a configuration issue for Aurora. And Wariorr, EarthMC developer, handles this also patches Towny and is happy with them being made here.

@Basically-Andrew
Copy link
Contributor Author

Should just be a change here.

@ghost
Copy link

ghost commented Jun 4, 2022

This doesn't seem to be a Towny bug as this didn't work on Nova. Appears to be a configuration issue for Aurora. And Wariorr, EarthMC developer, handles this also patches Towny and is happy with them being made here.

If it has to do with Towny and reproducible without it being on EarthMC, it's a Towny Bug. Simple as that.

Should just be a change here.

Close but this fix does not address the issue that sometimes nations may disable their public nation spawn (if enabled in Towny config to do so). I have completed the changes here: TownyAdvanced/Towny#5940

@LlmDl
Copy link

LlmDl commented Jun 4, 2022

This isn't a bug. The nation isn't public so they cannot be spawned to by a non-ally and non-member.

@Basically-Andrew
Copy link
Contributor Author

This isn't a bug. The nation isn't public so they cannot be spawned to by a non-ally and non-member.

This is still an issue. The message "That nation is not public." Will always display and confuse players when public spawn is disabled in config. If the server has public spawning disabled, a more appropriate message should display that would let the player know they aren't allied and should let their diplomat, chancellor, or nation leader ally the nation.

@LlmDl
Copy link

LlmDl commented Jun 4, 2022

Ah well your ticket doesn't say anything about a server having public spawning disabled.

@ghost
Copy link

ghost commented Jun 5, 2022

This issue will be resolved in the next Towny Pre-Release: TownyAdvanced/Towny@498b22f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants