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

LOL LMAO ROFL XD XD XP XP XP #145

Closed
wants to merge 2 commits into from
Closed

Conversation

demipixel
Copy link
Contributor

XP

@rom1504
Copy link
Member

rom1504 commented Dec 6, 2015

Can you add doc for these functions ? (so we don't have to add lot and lot of docs later)

rom1504 added a commit that referenced this pull request Dec 7, 2015
@rom1504
Copy link
Member

rom1504 commented Dec 7, 2015

Oh well, I kind of failed my merge, but all the commits are in master now.

@rom1504 rom1504 closed this Dec 7, 2015
@demipixel demipixel deleted the xp branch December 7, 2015 02:51
@ghost
Copy link

ghost commented Dec 7, 2015

@rom1504, we should really start thinking about a way to store all of this stuff like a function that stores key-values? It would be good to start saving info on players.

@rom1504
Copy link
Member

rom1504 commented Dec 7, 2015

@mhsjlw yes, but we probably want to start by using the vanilla storage format

@ghost
Copy link

ghost commented Dec 7, 2015

Do we really need to do that? I think it would make more sense maybe just making a wrapper around an sqlite3 server that starts when flying-squid does, then just make methods for storing data in the database. Then, if we wanted to we could make a converter that converts the database to world format and so on so forth. Also, we could add abilities to add different databases other than sqlite3 like PostgreSQL and MongoDB and MySQL

@rom1504
Copy link
Member

rom1504 commented Dec 7, 2015

@mhsjlw I think we should be able to use different backend, but the default should be vanilla.

But if you want to start working on a non-vanilla backend, feel free to ;)

@rom1504
Copy link
Member

rom1504 commented Dec 7, 2015

also see PrismarineJS/prismarine-world#11

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.

2 participants