owenashurst / agar.io-clone

Agar.io clone written with Socket.IO and HTML5 canvas
MIT License
2.88k stars 1.12k forks source link

Database integration #471

Closed CoderJoeW closed 7 years ago

CoderJoeW commented 7 years ago

It took me like 5-6 hours of experimenting and bug fixes but I FINALLY got it working so a breakdown

I added sql settings into config.json so that people can modify to their needs easily

Added code to server to connect to database based on settings in config.json

Added code to log incorrect password input for admin (it was on the todo list)

This could really be useful for the future as well with a level/xp system I would start working on it but....I just spent like 5 hours on all this.....coffee and code || sleep

   

CoderJoeW commented 7 years ago

:shipit:

CoderJoeW commented 7 years ago

Anybody looking at this?

vikramdurai commented 7 years ago

I am.

vikramdurai commented 7 years ago

:shipit:

Max1Truc commented 7 years ago

:shipit:

huytd commented 7 years ago

👍 Thanks everybody! Merging!

Max1Truc commented 7 years ago

cool !

TheDiamondYT1 commented 6 years ago

@huytd You really need to learn to use the squash and merge button.

huytd commented 6 years ago

Yes master 🙇

LOL, just kidding, I don't think Github has that button (at least, at the time I hit the merge), but yes, I should do it on the command line for big pull requests like this.

👍

TheDiamondYT1 commented 6 years ago

@huytd screenshot_2017-09-11-18-46-09