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

Add in-game chat #245

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Add in-game chat #245

wants to merge 6 commits into from

Conversation

lhvy
Copy link
Contributor

@lhvy lhvy commented Sep 23, 2021

Resolves #56

Adds a chat into the lobbies. Currently WIP, works but needs proper placement and styling.
Screen Shot 2021-09-24 at 00 40 56

I'll mark the PR as a draft until I clean up the UI. Any feedback would be appreciated! 🙂

@Lattyware
Copy link
Owner

Cool contribution! Looks like a solid idea. I'd recommend keeping it as simple as possible (e.g: not keeping history on the server, just relay to clients and forget) to avoid server load and not get too bogged down in it. I'll take a proper look when I get a chance, but nothing jumps out at me taking a quick look.

@lhvy
Copy link
Contributor Author

lhvy commented Nov 29, 2021

I just noticed the activity over the past 24 hours, thanks for merging #243! I've been quite busy for the past few months and haven't had any time to finish up this PR. I'll hopefully have time for it in the next few weeks...

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.

In-game chat.
2 participants