Add Lodestones (rebased) #2154

Merged
cora merged 8 commits from chmodsayshello/MineClone2:lodestone_rebased into master 2022-05-01 02:38:52 +02:00
Member

This is the rebased version of #2138, for more informations about Lodestones see https://minecraft.fandom.com/wiki/Lodestone

This is the rebased version of #2138, for more informations about Lodestones see https://minecraft.fandom.com/wiki/Lodestone
chmodsayshello added 7 commits 2022-04-30 11:04:56 +02:00
Contributor

quickly testing ... still works but there's a stray debug message "cooldown not over yet".

quickly testing ... still works but there's a stray debug message "cooldown not over yet".
Author
Member

quickly testing ... still works but there's a stray debug message "cooldown not over yet".

Oh, Sorry, I forgot to remove it, everytime it tries to force load (minetest.get_node().name == "ignore") the lodestone it checks if it's been 3 mins since the last one, and then it either logs "cooldown not over yet" or "forceloaded, cooldown reset!"

> quickly testing ... still works but there's a stray debug message "cooldown not over yet". Oh, Sorry, I forgot to remove it, everytime it tries to force load (`minetest.get_node().name == "ignore"`) the lodestone it checks if it's been 3 mins since the last one, and then it either logs "cooldown not over yet" or "forceloaded, cooldown reset!"
chmodsayshello added 1 commit 2022-04-30 12:02:09 +02:00
Contributor

Oh, Sorry, I forgot to remove it, everytime it tries to force load (minetest.get_node().name == "ignore") the lodestone it checks if it's been 3 mins since the last one, and then it either logs "cooldown not over yet" or "forceloaded, cooldown reset!"

yeah i figured no need to spam the player with it ^^

thanks for quick fix. i'll test a bit more after work then merge if all works out

> Oh, Sorry, I forgot to remove it, everytime it tries to force load (`minetest.get_node().name == "ignore"`) the lodestone it checks if it's been 3 mins since the last one, and then it either logs "cooldown not over yet" or "forceloaded, cooldown reset!" yeah i figured no need to spam the player with it ^^ thanks for quick fix. i'll test a bit more after work then merge if all works out
chmodsayshello added the
nodes
Minecraft >= 1.13
labels 2022-04-30 13:22:20 +02:00
cora approved these changes 2022-05-01 02:38:37 +02:00
cora left a comment
Contributor

Works nicely <3

Works nicely <3
cora merged commit 38f1db7a04 into master 2022-05-01 02:38:52 +02:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: VoxeLibre/VoxeLibre#2154
No description provided.