Skip to content

Show progress bar without commands enabled#482

Open
brandyballoon wants to merge 4 commits into
pop4959:masterfrom
brandyballoon:progress-bar
Open

Show progress bar without commands enabled#482
brandyballoon wants to merge 4 commits into
pop4959:masterfrom
brandyballoon:progress-bar

Conversation

@brandyballoon

Copy link
Copy Markdown

No description provided.

@pop4959

pop4959 commented Jun 27, 2026

Copy link
Copy Markdown
Owner

Hey, thanks for the PR! I'm assuming you're taking a look at #450 and #479 with this?

Just removing the permission check here entirely isn't correct though, it should be checking for the player's permissions instead of gamemaster (which defaults to gamemaster if a permission mod isn't installed), or if the server is running in Singleplayer mode. You can test this with LuckPerms.

@pop4959 pop4959 added enhancement New feature or request fabric Issues related to the Fabric platform forge Issues related to the Forge platform neoforge Issues related to the NeoForge platform labels Jun 27, 2026
@brandyballoon

Copy link
Copy Markdown
Author

OK sorry it was a quick hack without fully understanding how it works. Yes I found #450 but didn't look at #479 yet. Thanks for the feedback.

@brandyballoon

Copy link
Copy Markdown
Author

If I were to have a go at doing this properly, what is the desired behavior for the remote players when a single player world is opened to LAN with commands off?

@pop4959

pop4959 commented Jun 27, 2026

Copy link
Copy Markdown
Owner

If I were to have a go at doing this properly, what is the desired behavior for the remote players when a single player world is opened to LAN with commands off?

As I described above, but as an example the /chunky command permissions currently work the same way the boss bar is intended to, so you can have a look at that for an example. It's a relatively minor change either way. If you're not able to figure it out, no worries because I was planning on making this change soon anyways when I have a chance to go through the issue backlog.

@brandyballoon

Copy link
Copy Markdown
Author

How does it look now? Progress bar should now always show in single player mode (Fabric, Forge, NeoForge), and in multiplayer only if the player can run chunky commands or has chunky.progressbar (Fabric + LuckPerms) permission.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request fabric Issues related to the Fabric platform forge Issues related to the Forge platform neoforge Issues related to the NeoForge platform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants