MehVahdJukaar / WoodGood

Other
54 stars 33 forks source link

TableTopCraftModule "addTile" fix #628

Closed andrew0030 closed 1 month ago

andrew0030 commented 1 month ago

I tweaked the TableTopCraftModule code a bit, it now utilizes the getModTile method instead of loader specific code. This is necessary because otherwise the game crashes, it may or may not have worked for just fabric, however given this is the common module, the intention is clearly for it to run on all loaders.

I also adjusted the "supported" Table Top Craft versions comment, it now properly indicates which versions are functional, (all of them except one 👏).

I tested these changes in production on both fabric and forge with a big version range of table top craft.

andrew0030 commented 1 month ago

Ah I knew I forgot something, would be nice if you could also add Table Top Craft to the Supported Mods list.

Xelbayria commented 1 month ago

Will do!

All looks good! Merging it