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

Create an Octane upgrade strategy guide #850

Merged
merged 7 commits into from
Jun 13, 2019
Merged

Create an Octane upgrade strategy guide #850

merged 7 commits into from
Jun 13, 2019

Conversation

jenweber
Copy link
Contributor

@jenweber jenweber commented Jun 9, 2019

Closes #849

@jenweber jenweber requested a review from wycats June 9, 2019 21:46
@jenweber
Copy link
Contributor Author

jenweber commented Jun 9, 2019

@vaidehijoshi - I can't add you as a formal reviewer in the GitHub interface, but please review if you can!

@jenweber jenweber requested a review from pzuraq June 10, 2019 01:54
guides/release/upgrading/index.md Outdated Show resolved Hide resolved
guides/release/upgrading/index.md Outdated Show resolved Hide resolved
guides/release/upgrading/index.md Outdated Show resolved Hide resolved
guides/release/upgrading/index.md Outdated Show resolved Hide resolved
The best way to discover new features is to read the [release blog posts](https://blog.emberjs.com/tags/releases.html).
If a new feature requires you to opt-in, it's called an optional feature.
Follow [this guide](../../configuring-ember/optional-features/) to learn which optional features are available in your app's version and how to enable them.
In many cases, codemods will be available to help you make syntax-related updates.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be useful to define what a codemod is here, and when to use them.

### Refactoring checklist

For many of the optional features, the thing they affect the most is what you see in newly created files, not your existing code.
That's a good place to start, but eventually you may want to refactor existing code so that your app follows one main programming model, not a mixture of Octane and Classic.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we mention something here about how, even though your app will keep working on Octane even though it isn't completely idiomatic, you probably want to follow Octane idioms (and hence it's good to refactor in that direction to use those new idioms)?

There's no one-size-fits-all strategy, but here is a checklist you can adapt, once you're familiar with what Octane has to offer:

1. Convert curly bracket components (`{{my-component}}`) to Angle Brackets (`<MyComponent />`). They are a normal, out-of-the-box feature of Ember since [3.4](https://blog.emberjs.com/2018/10/07/ember-3-4-released.html) that does not change a component's behavior. Follow the [Angle Bracket Syntax guide](../../reference/syntax-conversion-guide) for examples.
2. Use Named Arguments and `this` in your templates. This also does not change component behavior.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should give an example of Named Arguments (or define it?) here

guides/release/upgrading/index.md Outdated Show resolved Hide resolved
guides/release/upgrading/index.md Outdated Show resolved Hide resolved
guides/release/upgrading/index.md Outdated Show resolved Hide resolved
@vaidehijoshi
Copy link
Contributor

Nice work! My comments are pretty much all just minor suggestions to improve readability and some ideas of where we can add definitions/context. 😄

Copy link
Contributor

@wycats wycats left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving as a good first pass

@jenweber jenweber merged commit da9318c into octane Jun 13, 2019
@jenweber jenweber deleted the upgrading-intro branch June 13, 2019 18:11
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

Successfully merging this pull request may close these issues.

3 participants