Skip to content

add mod respawn - #473

Merged
BuckarooBanzay merged 1 commit into
minetest-mapserver:masterfrom
bgstack15:dev/support-respawn
Sep 15, 2026
Merged

BuckarooBanzay merged 1 commit into
minetest-mapserver:masterfrom
bgstack15:dev/support-respawn

Conversation

@bgstack15

Copy link
Copy Markdown
Contributor

This adds a new feature: support mod respawn places database.

The admin can configure the color of the respawn "places." There are a few other data in mod respawn, which might be implemented on the map in the future, e.g., per-team spawn points, which would benefit from additional color settings.

This bundles in the commits from #468 which is about Mineclonia signs, but if necessary, that can be extracted.

AI disclosure: approximately 10% of this code was written by AI:

  • mcl_sign.go decodeUText()
  • RespawnOverlay.js escapeHtml()

@BuckarooBanzay

Copy link
Copy Markdown
Member

You can ignore the jshint issues, the linter/hinter is outdated from the looks of it:

map/overlays/RespawnOverlay.js: line 13, col 20, 'async functions' is only available in ES8 (use 'esversion: 8').
map/overlays/RespawnOverlay.js: line 56, col 7, Missing semicolon.
map/overlays/RespawnOverlay.js: line 79, col 11, 'async functions' is only available in ES8 (use 'esversion: 8').

AI disclosure: approximately 10% of this code was written by AI:

Nothing wrong with that, if it gets the job done 🤷

Also: thanks for the caching-fixes in the Dockerfile 👍

@BuckarooBanzay BuckarooBanzay added the enhancement New feature or request label Sep 15, 2026
@BuckarooBanzay
BuckarooBanzay merged commit 2bcdf65 into minetest-mapserver:master Sep 15, 2026
3 of 4 checks passed
@bgstack15

Copy link
Copy Markdown
Contributor Author

I guess I overlooked that I had manipulated the docker file. That was for my own benefit while iterating, but it probably would help anybody else trying to build.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants