underscoreio / essential-slick

Essential Slick Pandoc Source
https://underscore.io/books/essential-slick/
39 stars 8 forks source link

Corrections and comments from Joseph Ottinger 20 Nov 2015 #87

Closed d6y closed 8 years ago

d6y commented 8 years ago

From: Joseph Ottinger https://github.com/jottinger Source: https://gitter.im/underscoreio/scala?at=564f473654ba2ca242608dc5

A few other minor notes as I go through: chapter 1's hello world exercise didn't mention importing duration (very minor thing);

also, 1.3.7 generated a TON of output that I didn't expect (but which made perfect sense), and also didn't use Future where I expected it to (the second example)

however, I actually tried to go through the Slick hello world online, and failed dramatically - this has worked like a charm out of the box (from github and from the chapter tutorial itself), so that's very good. I understand a lot more than I did, already.

It would be nice to see how to generate the schema if it's not already present, though (just as a convenience for multiple runs)

wearing my editor hat for a moment (as editor for TheServerSide and Apress): section 2.3 says "Querys have access to the type of the Table used to create them, allowing us to directly reference the columns on the Table when we’re using combinators like map and filter." "Querys" has "Query" as code (since it's referring to a type) - but would be better written as "Query instances" to prevent pluralizing the class type.

d6y commented 8 years ago

Section 3.1 is really nice, BTW - but I'd prefer more verbose text. I personally far far far prefer fully runnable code to snippets that highlight specific features - the fully running code can have important parts pointed out easily enough

jonoabroad commented 8 years ago
d6y commented 8 years ago

I've patched in the missing imports. We're done here I think.