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

Deprecation guide for jQuery removal #255

Merged
merged 4 commits into from
Jan 23, 2019

Conversation

simonihmig
Copy link
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
content/ember/v3/jquery-apis.md Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
@simonihmig
Copy link
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
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
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?

content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
content/ember/v3/jquery-apis.md Outdated Show resolved Hide resolved
@rwjblue
Copy link
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...

@simonihmig
Copy link
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
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