Skip to content

Conversation

@Askaholic
Copy link
Collaborator

Fixes #323

Not exactly sure how the client handles newlines but presumably this will look better than it did before.

@Askaholic Askaholic changed the title Clean up spacing and <br>'s Issue/#323 Clean up spacing and <br>'s Dec 22, 2018
@norraxx
Copy link
Collaborator

norraxx commented Dec 24, 2018

If you want, we can test it on test environment.

@Askaholic
Copy link
Collaborator Author

Sure. I set up a build using faf-stack locally, but haven't figured out how I can point my client at it yet for testing.

@norraxx
Copy link
Collaborator

norraxx commented Dec 24, 2018

I'm unavailable for 3 days, ask @Rackover for help. Simpliest way how to check it, is build image from your branch on test.

@Rackover
Copy link
Member

Hey, will have no working python server environment until january, but you can ping me on slack and i'll help you set up one if you need.

@Rackover Rackover merged commit 6505091 into FAForever:develop Dec 25, 2018
@Askaholic Askaholic deleted the issue/#323-spacing branch December 29, 2018 23:23
Brutus5000 pushed a commit that referenced this pull request Jun 24, 2019
* Clean up spacing

* send_warning sends <br/> instead of <br>
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