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

blog(embertimes135): EmberMap Native Decorator #525

Merged
merged 2 commits into from
Feb 11, 2020

Conversation

chrisrng
Copy link
Member

No description provided.

@probot-autolabeler probot-autolabeler bot added the embertimes PR for an Ember Times issue label Feb 10, 2020

<add your name to author list, top and bottom>
<add blurb and emoji to "SOME-INTRO-HERE">
With Ember 3.10, all the core APIs and surrounding ecosystem libraries can start adopting decorators as per the [stage 1 implementation](https://github.com/tc39/proposal-decorators) which would pave the way for ES classes in Ember. However, the happy path is still to wait until [the Octane release or 3.15](https://blog.emberjs.com/2019/12/20/ember-3-15-released.html) to start using both ES classes and decorators together for all of your Ember app code.
Copy link
Member

Choose a reason for hiding this comment

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

I haven't watched the video yet, so in this paragraph, I was a little confused by what you had meant to convey regarding Ember 3.10 versus 3.15.

Copy link
Member Author

Choose a reason for hiding this comment

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

they talked about the history – im gonna submit for now since ill be away from laptop lol but feel free to rewrite as needed :) thanks for the review!

Copy link
Member

@ijlee2 ijlee2 left a comment

Choose a reason for hiding this comment

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

Looks good, Chris. I wasn't sure about what your 2nd paragraph meant, but it might be just me not having watched the video or read the RFC 440 yet. If you're busy, I think it's okay to leave the paragraph as is.

@chrisrng chrisrng merged commit b60069d into blog/embertimes-135 Feb 11, 2020
@chrisrng chrisrng deleted the chrisrng-patch-1 branch February 11, 2020 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
embertimes PR for an Ember Times issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants