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

Database integration #471

Merged
merged 67 commits into from
Mar 23, 2017
Merged

Database integration #471

merged 67 commits into from
Mar 23, 2017

Conversation

CoderJoeW
Copy link
Contributor

@CoderJoeW CoderJoeW commented Nov 25, 2016

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

  •  General Comment
  • :shipit:
  • Anybody looking at this?
  • I am.
  • :shipit:
  • :shipit:
  • Hey, huytd, are you looking at this ?
    It's OKAY !
  • 👍 Thanks everybody! Merging!
  • cool !

   

@CoderJoeW
Copy link
Contributor Author

:shipit:

@CoderJoeW
Copy link
Contributor Author

Anybody looking at this?

@vikramdurai
Copy link
Contributor

I am.

@vikramdurai
Copy link
Contributor

:shipit:

Copy link

@Max1Truc Max1Truc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool ! I checked this and all is good ! No problem !

@Max1Truc
Copy link

:shipit:

@huytd
Copy link
Collaborator

huytd commented Mar 23, 2017

👍 Thanks everybody! Merging!

@huytd huytd merged commit b2603e4 into owenashurst:master Mar 23, 2017
@Max1Truc
Copy link

cool !

@TheDiamondYT1
Copy link

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

@huytd
Copy link
Collaborator

huytd commented Sep 11, 2017

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

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

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.

5 participants