-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
General Feature Additions #577
Conversation
…ws all homes not just private homes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your work, looks pretty good! :) Appreciate it.
In addition to the review comments, could you propegate your locale changes to the other locale langs?
bukkit/src/main/java/net/william278/huskhomes/user/BukkitUser.java
Outdated
Show resolved
Hide resolved
@@ -110,6 +112,8 @@ private Optional<PaginatedList> generateList(@NotNull CommandUser executor, @Not | |||
plugin.getLocales() | |||
.getRawLocale("home_list_item", | |||
Locales.escapeText(home.getName()), home.getSafeIdentifier(), | |||
home.isPublic() ? plugin.getLocales().getRawLocale("home_is_public").orElse("�fb9a&Public"): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's just supply Public
or Private
if no locale is found here without formatting :)
common/src/main/java/net/william278/huskhomes/user/OnlineUser.java
Outdated
Show resolved
Hide resolved
…here it would not remove invulnerability from player on sponge
Done all changes |
Implements Add configurable invulnerability period after teleporting
Implements Show home privacy status in /homelist