-
Notifications
You must be signed in to change notification settings - Fork 30
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
Adding guestbook-datomic PR #6
Conversation
Not sure if you wanted to make changes to it =)... |
Hmm, what types of things would you want me to document? A high level idea is sufficient, I can start to organise something and see what I come up with =)... |
I was thinking it might be good to give a brief overview of getting up and running with Datomic. Something similar to the existing material in the "Database Access" section, and we could link the sample project from there. If you can think of any gotchas to mention, stuff like using mem db for dev, etc. would be good to add. |
Ok, so sections on:
For I'm not sure it's super robust, but for anyone wanting to run a datomic instance in the heroku ecosystem at a reasonable starting price point I've found it works great. I'm pretty sure that I can then work out how to migrate up if I need it, but for the moment, its sufficient. Do you want me to do a section on it? |
Yeah these sections sound pretty reasonable to help people get started. I'd focus on documenting things specific to using Datomic from Luminus, and link to the official docs for more details on Datomic itself. Giving an example of the Heroku setup would be great as well. I agree that it's generally a good idea not to expose the DB to the outside world. :) |
@yogthos I've got something drafted for 2), shall I pop it up somewhere? Maybe a PR on the docs?. For 1) I'm still working out what to write regarding it, and for 3) I've got something working, but there's still an open issue I'm trying to get sorted out. I can document what I have working already, or hopefully I'll hear something soon and can get it fixed. |
Yeah, a PR for the docs site would be ideal, and probably wouldn't hurt to add what you've got already. Can always do another PR later once the issue resolves. |
👍 |
PR associated with #5.