Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update JukeAlertLogger.java #37

Merged
merged 1 commit into from
Aug 12, 2015
Merged

Update JukeAlertLogger.java #37

merged 1 commit into from
Aug 12, 2015

Conversation

TealNerd
Copy link

Fixes issue #25

@ttk2
Copy link

ttk2 commented Aug 12, 2015

what does this do with long playernames now?

@TealNerd
Copy link
Author

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 added a commit that referenced this pull request Aug 12, 2015
Update JukeAlertLogger.java
@ttk2 ttk2 merged commit 63e2018 into Civcraft:master Aug 12, 2015
@ttk2
Copy link

ttk2 commented Aug 12, 2015

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

@TealNerd
Copy link
Author

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
#37 (comment).

@ttk2
Copy link

ttk2 commented Aug 12, 2015

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
Copy link
Author

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
Copy link

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

@ttk2
Copy link

ttk2 commented Aug 12, 2015

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
Copy link
Author

biggestnerd actually, i made tealnerd so nobody could impersonate me

@ttk2
Copy link

ttk2 commented Aug 13, 2015

credentials sent.

@ttk2
Copy link

ttk2 commented Aug 16, 2015

um so did this end up happening?

@TealNerd
Copy link
Author

its working for me on civtest

@ttk2
Copy link

ttk2 commented Aug 16, 2015

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

rourke750 pushed a commit that referenced this pull request Aug 12, 2018
Show a message on snitch destruction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants