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

Updates to Blogger #1225

Open
case-eee opened this issue Oct 13, 2016 · 1 comment
Open

Updates to Blogger #1225

case-eee opened this issue Oct 13, 2016 · 1 comment

Comments

@case-eee
Copy link
Contributor

  • Line 98: Does this mean that our students will have to continue using bin/rails if they have Rails 5 installed?
  • Line 100: Do we want to switch this to a rails g migration and have them create the model by hand?
  • Line 119: Do we want to add a line to emphasize that if they change this method and then try to rollback that rake won't know how to undo the migration? We mention that there used to be an up and a down, but don't really walk them through the implications of that.
  • Line 272: Maybe not something to do now, but seems like we might want to take out the generate controller.
  • Line 575: Is this stale? I don't see model_name referenced in the error.

@s-espinosa

@berndbausch
Copy link

I experience the same mild confusion when reading "Huh? We didn’t call a method model_name?" in section I1. There is no model_name anywhere.

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

2 participants