Add common protection api support & small fixes

Ladysnake / BLAST

A simple Minecraft bomb mod for Fabric.

https://ladysnake.github.io/wiki/blast.html

13 stars 14 forks source link

Add common protection api support & small fixes #43

Closed sleepy-evelyn closed 10 months ago

sleepy-evelyn commented 10 months ago

Added support for Patbox's common protection API (https://github.com/Patbox/common-protection-api) which prevents explosions bypassing claims for land claiming mods like Cadmus (https://modrinth.com/mod/cadmus)

Also fixes #44 and an issues where you could get floating fire blocks on the client

sleepy-evelyn commented 10 months ago

Will review this more in depth later but at first glance it's good work again - just makes me think we should really refactor the block classes at some point to reduce the code duplication

True. That's gonna take a bit longer I think. Will of course be worth switching to variables that mean something as well

sleepy-evelyn commented 10 months ago

Video of it working: https://youtu.be/6LrdmD8crsg

sleepy-evelyn commented 10 months ago

Added documentation to the readme and sent a PR for the wiki: https://github.com/Ladysnake/Ladysnake.github.io/pull/77. Unless any issues or changes are required should be finished with this PR now