-
Notifications
You must be signed in to change notification settings - Fork 468
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
Active Record tutorial #1234
Active Record tutorial #1234
Conversation
Generally looking good! except usually its spelled ActiveRecord, not Active Record Review status: 0 of 8 files reviewed at latest revision, 8 unresolved discussions. build-a-ruby-app-with-cockroachdb-activerecord.md, line 3 at r1 (raw file):
ActiveRecord* build-a-ruby-app-with-cockroachdb-activerecord.md, line 12 at r1 (raw file):
ActiveRecord* build-a-ruby-app-with-cockroachdb-activerecord.md, line 14 at r1 (raw file):
ActiveRecord* build-a-ruby-app-with-cockroachdb-activerecord.md, line 24 at r1 (raw file):
This should be spelled ActiveRecord, not Active Record, throughout. build-a-ruby-app-with-cockroachdb-activerecord.md, line 34 at r1 (raw file):
ActiveRecord* Remove the build-a-ruby-app-with-cockroachdb-activerecord.md, line 89 at r1 (raw file):
ActiveRecord* build-a-ruby-app-with-cockroachdb.md, line 12 at r1 (raw file):
ActiveRecord* build-an-app-with-cockroachdb.md, line 15 at r1 (raw file):
ActiveRecord Comments from Reviewable |
2cdf315
to
bafa977
Compare
Review status: 0 of 8 files reviewed at latest revision, 8 unresolved discussions. build-a-ruby-app-with-cockroachdb-activerecord.md, line 3 at r1 (raw file): Previously, jordanlewis (Jordan Lewis) wrote…
Done. build-a-ruby-app-with-cockroachdb-activerecord.md, line 12 at r1 (raw file): Previously, jordanlewis (Jordan Lewis) wrote…
Done. build-a-ruby-app-with-cockroachdb-activerecord.md, line 14 at r1 (raw file): Previously, jordanlewis (Jordan Lewis) wrote…
Done. build-a-ruby-app-with-cockroachdb-activerecord.md, line 24 at r1 (raw file): Previously, jordanlewis (Jordan Lewis) wrote…
Done. build-a-ruby-app-with-cockroachdb-activerecord.md, line 34 at r1 (raw file): Previously, jordanlewis (Jordan Lewis) wrote…
Done. build-a-ruby-app-with-cockroachdb-activerecord.md, line 89 at r1 (raw file): Previously, jordanlewis (Jordan Lewis) wrote…
Done. build-a-ruby-app-with-cockroachdb.md, line 12 at r1 (raw file): Previously, jordanlewis (Jordan Lewis) wrote…
Done. build-an-app-with-cockroachdb.md, line 15 at r1 (raw file): Previously, jordanlewis (Jordan Lewis) wrote…
Done. Comments from Reviewable |
bafa977
to
5a78a9a
Compare
Once we commit to supporting additional ORMs, we can add this bit back in.
Expand Ruby "build an app" tutorial to offer usage of pg driver or Active Record ORM.
Fixes #547
This change is