Civcraft / JukeAlert

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

Add cart breaks to snitch logging #38

Closed Maxopoly closed 8 years ago

Maxopoly commented 8 years ago

I think I got everything that needs to be changed to add another action and event, but someone with insight into JA should really review this.

ttk2 commented 8 years ago

looks cool, sorry with jenkins down its been hard to roll out your other stuff, trying to fix that now .

ttk2 commented 8 years ago

@ProgrammerDan review when you get a chance.

ttk2 commented 8 years ago

so is this ready to go?

Maxopoly commented 8 years ago

No, I just need to make the changes Erocs suggested quickly

ttk2 commented 8 years ago

no rush, thanks for the work!

Maxopoly commented 8 years ago

Alright, should be good now

ttk2 commented 8 years ago

on civtest, report when tested.

ttk2 commented 8 years ago

well not on Civtest

ttk2 commented 8 years ago

@rourke750 jukealert builds are failing, looks like itemexchange. Do I just need to put an updated .jar somewhere?

http://build.civcraft.co/job/JukeAlert-master/201/console

Maxopoly commented 8 years ago

Bump

What's the status of this?

ttk2 commented 8 years ago

this should be good to go online, I will stage it when I get a chacne.

Maxopoly commented 8 years ago

This was never tested, but then again even if it doesn't work, it shouldn't break anything else. Alright.

https://www.youtube.com/watch?v=eUFY8Zw0Bag