Skip to content

Deprecation guide for jQuery removal#255

Merged
rwjblue merged 4 commits intoember-learn:masterfrom
simonihmig:jquery-apis
Jan 23, 2019
Merged

Deprecation guide for jQuery removal#255
rwjblue merged 4 commits intoember-learn:masterfrom
simonihmig:jquery-apis

Conversation

@simonihmig
Copy link
Copy Markdown
Contributor

As per RFC386

Please check twice, as I am not a native speaker! 😉
Also whether the message/tone seems appropriate! /cc @rwjblue @tomdale

locks
locks previously requested changes Jan 19, 2019
@simonihmig
Copy link
Copy Markdown
Contributor Author

@locks updated after your review, thanks!

since: '3.8'
---

As jQuery will be removed from Ember by default, APIs that are coupled to jQuery have been deprecated.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
As jQuery will be removed from Ember by default, APIs that are coupled to jQuery have been deprecated.
As of Ember 3.2.0, Ember no longer requires that all applications include jQuery, therefore APIs that are coupled to jQuery have been deprecated.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I believe making jQuery optional was implemented for Ember.js 3.3, wasn't it?
But there was another thing, IIRC the 3.3 release of Ember CLI was postponed (making it 3.4), which included the necessary changes required by the (first) RFC. So we referred to 3.4 as the version which made jQuery optional, as this was the least confusing option for users (who usually have Ember & CLI in lockstep). This is at least what the Guides say: https://guides.emberjs.com/release/configuring-ember/optional-features/#toc_jquery-integration

So I changed this to 3.4.0 now, hope this is fine?

@rwjblue
Copy link
Copy Markdown
Contributor

rwjblue commented Jan 22, 2019

The changes I suggested are actually pretty minor edits to the original prose, looks like a bunch more when viewed here...

Co-Authored-By: simonihmig <ihmig@kaliber5.de>
@simonihmig
Copy link
Copy Markdown
Contributor Author

The changes I suggested are actually pretty minor edits to the original prose, looks like a bunch more when viewed here...

@rwjblue Thanks! Applied your suggestions, reads much better now. Just one remark left, see my other comment...

@simonihmig
Copy link
Copy Markdown
Contributor Author

Oh, and just updated the since meta value to 3.9, since this is the version the newly merged deprecations will be released. @rwjblue confirm?

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