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

Minor typos and fixes #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions docs/blog-post.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export default Model.extend({
});
```

What is that funky syntax? `import Model from 'ember-data/model` and `export default Model.extend()`? Welcome to the world of tomorrow!
What is that funky syntax? `import Model from 'ember-data/model'` and `export default Model.extend()`? Welcome to the world of tomorrow!

Those `import` and `export` statements use ECMAScript 6 module syntax. Thanks to the magic of transpilers, we can already use them today even though no browsers support ES6 yet. This should look familiar if you have used Node.js or AMD modules, there's just slightly different syntax. We're importing a module from 'ember-data/model' and calling it `Model`. Then we're extending the `Model` class and using that as our module export.

Expand Down Expand Up @@ -288,7 +288,7 @@ Based on our route's dynamic URL segments the REST Adapter will make the proper

**ProTip™** The store action determines the model name based on the defined dynamic segment. In our example `:blog_post_id` contains the proper snake-case name for our model with the suffix `_id` appended.

To confirm that this works, **delete the `app/routes/blog-post.js` file** and verify that our blog post page (http://localhost:4200/post/1) still works properly after reload.
To confirm that this works, **delete the `app/routes/blog-post.js` file** and verify that our blog post page (`http://localhost:4200/post/1`) still works properly after reload.

Ember-CLI created a route test file automatically as well. To make sure your tests continue to pass, if you delete `app/routes/blog-post.js` you should also delete `tests/unit/routes/blog-post-test.js`

Expand Down
4 changes: 2 additions & 2 deletions docs/next.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
# Feedback

https://hbrysiewicz.typeform.com/to/DMKtHS
[https://hbrysiewicz.typeform.com/to/DMKtHS](https://hbrysiewicz.typeform.com/to/DMKtHS)

Please take 2 minutes to complete our breif workshop feedback form!
Please take 2 minutes to complete our brief workshop feedback form!


# Where do I go next?
Expand Down