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

Add persistance #16

Open
xmclark opened this issue Dec 8, 2018 · 1 comment
Open

Add persistance #16

xmclark opened this issue Dec 8, 2018 · 1 comment

Comments

@xmclark
Copy link
Contributor

xmclark commented Dec 8, 2018

There needs to be a pattern for creating routes that interact with our data.

We don't have many restrictions on the technology for our persistence layer. Home servers have a mixture of normal entity records and event sources. It does not need to be high performance right now.

We can optimize for developer experience and ease of use for now, and optimize later (maybe swap out database technologies).

@xmclark
Copy link
Contributor Author

xmclark commented Dec 10, 2018

#19 adds some basic support for sqlite with diesel. This approach should be acceptable for now.

That PR does not add a way to make integration tests but we may be able to add them later. See these links:
https://github.com/actix/actix-website/blob/master/content/docs/testing.md#integration-tests
https://doc.rust-lang.org/rust-by-example/testing/integration_testing.html

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

1 participant