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

Clean up Source Code #4

Closed
mastergberry opened this issue Apr 25, 2014 · 1 comment
Closed

Clean up Source Code #4

mastergberry opened this issue Apr 25, 2014 · 1 comment

Comments

@mastergberry
Copy link
Contributor

~depends #3

This project seems really cool. There are a few things that would make the code as cool as the idea is. Take some time to improve on the GUI for the client to have a starting screen to select a server and port you wish to connect to. This way the project is not reliant on CSIL and port 4242.

In the second GUI screen when interacting with the actual application there should be a menu drop down or a button somewhere to disconnect from the server which will lead them back to the initial screen so they can log in to a new server.

Remove the hardcoded server bindings (such as the port) and make it take parameters in from the command line to make the server application easier to deploy.

If possible split the client into smaller classes and add some Javadocs to them to make this code clean and perfect.

~Estimated 300 points

@rhelmot
Copy link
Contributor

rhelmot commented Jan 21, 2018

Duplicates of #5 #9

@rhelmot rhelmot closed this as completed Jan 21, 2018
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

No branches or pull requests

2 participants