Skip to content
This repository has been archived by the owner on May 28, 2024. It is now read-only.

Demonstrated more Vapor features, integrated optional database #42

Closed
wants to merge 14 commits into from
Closed

Conversation

NathanFlurry
Copy link
Contributor

Resolves #29, but does it in a way which does not require the user to set up a database. It should work fine without a database attached. I added endpoints at /hash, /board, /db-version, /magic, and /math.

Replaced the posts/ endpoint with math/ for more clarity. The posts/
endpoint is confused and makes it thing that it's running a database
operation or something.
Added a hash endpoint that demonstrates Vapor's hashing ability
- Added MySQL to package
- Added build.sh and generate-xcodeproj.sh for easily building and
generating projects while still using MySQL
Now it is easier to navigate the document. The instructions may be a
bit confusing now that they are out of order, but from a brief glance
it looks fine.
- Added mysq.json config files
- Added extra line after every JSON file
Added the ability to get the version of the database
Models are now setup and prepared
- Added posting posts
- Rendering mustache document
- Saving to database
@NathanFlurry
Copy link
Contributor Author

If someone (probably @tannernelson) could take a shot at making /board pretty, that'd be great. CSS and HTML will be the death of me.

return try Node(node: [
"id": id,
"text": text,
// "user": try user().get() // TODO: Shouldn't this work?
Copy link
Contributor

Choose a reason for hiding this comment

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

This should work. What was the error?

@tanner0101
Copy link
Contributor

Woohoo! This is awesome.

Just need to copy the .travis.yml from a MySQL repo.

Like here: https://github.com/vapor/mysql/blob/master/.travis.yml

@tanner0101
Copy link
Contributor

done in #43

@tanner0101 tanner0101 closed this Aug 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

2 participants