search

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

Update JukeAlertLogger.java #37

Closed TealNerd closed 8 years ago

TealNerd commented 8 years ago

Fixes issue #25

ttk2 commented 8 years ago

what does this do with long playernames now?

TealNerd commented 8 years ago

Allows them to be just a touch longer so they actually show up all the way. Also fixes a weird issue where the data was misaligned with the column names because column names were being set to 22 wide and the data was set to 20

ttk2 commented 8 years ago

merged, do you have a testing server account to put this online?

TealNerd commented 8 years ago

I don't even know what that is

On Wednesday, August 12, 2015, ttk2 notifications@github.com wrote:

merged, do you have a testing server account to put this online?

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/JukeAlert/pull/37#issuecomment-130384676.

ttk2 commented 8 years ago

ah, we can get you an ssh account on the new testing server such that you can ssh in and use it yourself. interested?

also this is on civtest now, bump this thread when its tested.

TealNerd commented 8 years ago

I would like that yeah. also i tested it to the best of my ability. i only have one account with a long name but its working for that at least

ProgrammerDan commented 8 years ago

perhaps we could do a call on the discourse & reddit for folks with long IGNs to sign in and help test

ttk2 commented 8 years ago

testing thread would be good, teal your just TealNerd on the form? I will send you a message with credentials to the testing server if so.

TealNerd commented 8 years ago

biggestnerd actually, i made tealnerd so nobody could impersonate me

ttk2 commented 8 years ago

credentials sent.

ttk2 commented 8 years ago

um so did this end up happening?

TealNerd commented 8 years ago

its working for me on civtest

ttk2 commented 8 years ago

I will stage it after Roruke figures out the Namelayer issue then.