search

Civcraft / Citadel

Do not open issues here; open them on the maintained fork @ DevotedMC
https://github.com/DevotedMC/Citadel
BSD 3-Clause "New" or "Revised" License
6 stars 23 forks source link

Fix for Civcraft/Citadel/#170 #176

Closed suirad closed 8 years ago

suirad commented 8 years ago

This fixes Civcraft/Citadel/#170. I have tested it myself on my local testing server.

One thing that wasn't noticed, was that this bug also affected lava. This pull addresses both.

CivcraftBot commented 8 years ago

Can one of the admins verify this patch? Type 'ok to test' to test.

ttk2 commented 8 years ago

This looking good to merge @programmerdan? On Mar 19, 2016 3:41 AM, "CivcraftBot" notifications@github.com wrote:

Can one of the admins verify this patch? Type 'ok to test' to test.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/Civcraft/Citadel/pull/176#issuecomment-198656435

Maxopoly commented 8 years ago

Looks good

suirad commented 8 years ago

ok to test

ProgrammerDan commented 8 years ago

ok to test

ProgrammerDan commented 8 years ago

should be good, let me know results of testing @suirad

suirad commented 8 years ago

I would say good to merge. I have found in testing it stops the exact issue reported.

ProgrammerDan commented 8 years ago

works for us