Adds Native Class Constructor Update write-up to blog#3417
Adds Native Class Constructor Update write-up to blog#3417jayjayjpg merged 2 commits intoemberjs:blog/embertimes#52from Mi6u3l:blog/embertimes#52
Conversation
jayjayjpg
left a comment
There was a problem hiding this comment.
This reads great and is well-researched! I left two comments about formatting and a question if we want to add another CTA at the end. Otherwise I'm all 👍
| ## [SECTION TITLE](section url) | ||
| ## [Native Class Constructor Update 🛠](https://github.com/emberjs/rfcs/pull/337) | ||
|
|
||
| There is currently an open RFC proposing to change the behavior of EmberObject's constructor. |
There was a problem hiding this comment.
We aimed to put anything that describes Ember objects, methods or similar things into code quotes (``) in the past - I think it would make sense to put any mentions of EmberObject into ``, too so, it'd become EmberObject.
What do you think?
| This would assign the properties _after_ all of the class fields for any subclasses have been assigned. | ||
|
|
||
| One thing worth mentioning is that EmberObject will likely be deprecated in the near future and that ideally for non-Ember classes (things that aren't Components, Services, etc.) users should drop EmberObject altogether and use native classes only. | ||
|
|
There was a problem hiding this comment.
I think it'd be cool to add a final call-to action as well, reminding people that they can comment on the RFC to help finalizing it - what do you think?
jayjayjpg
left a comment
There was a problem hiding this comment.
Thank you for another great write-up this week, Miguel ✨
Adds Native Class Constructor Update write-up to blog