JumpstartLab/curriculum — Issue #61: JSBlogger Notes from Jim
Author: jcasimir
Created: 2012-03-05 13:13:27+00:00
State: open

@jimweirich is working with JSBlogger this week and had these revision suggestions:

* Migration methods are now up/down, not self.up/self.down
* Routing table shown using older format
* No need for h() method anymore (in general)
* @article.comments.new attaches the new comment to the in-memory version of the article, goofing up the display. I changed it to Comment.new(:article_id => @article.id)
* In the tagging section, @ruby, technology@ didn't render correctly (actually, there were a lot of mis-renders involving markdown)
* Because the TagsController was generated with method names, the error is missing template, not undefined action.
* Paperclip migration: change AND self.down?, should be change or up/down, (2) and drop the self.
* Sass information is out of date: (1) don't need to install, comes with rails, (2) don't put in public/stylesheet/styles.sass, put in app/assets/stylesheets/styles.css.sass, (3) sass example uses deprecated syntax (e.g. ! and = for variables)
* application.html.haml seems a bit off, doesn't work well with the provided CSS; particularly the #container/#content stuff.
* Comment: Seems strange that our first introduction to validations is the password confirmation. (see email from Jeff)
* When adding an admin user, the url should be /bloggers/new, not /users/new
* Typo: BloggerSessios => BloggerSessions
* Blogger.all.size should be Blogger.count