Skip to content
This repository has been archived by the owner on Mar 22, 2019. It is now read-only.

Missing Deprecations in Deprecations Guide #2256

Closed
60 tasks done
Gaurav0 opened this issue Jul 15, 2015 · 22 comments
Closed
60 tasks done

Missing Deprecations in Deprecations Guide #2256

Gaurav0 opened this issue Jul 15, 2015 · 22 comments
Assignees

Comments

@Gaurav0
Copy link
Contributor

Gaurav0 commented Jul 15, 2015

Please add to the deprecations guide here: http://emberjs.com/deprecations/v1.x/.

These are all of the deprecations in 1.13.x that need to be covered:

Thank you!

@wifelette
Copy link
Member

I just went through all the minutes on the list here, added the weeks that were missing (as in, happened after this initial issue was filed) and pulled in all the specific details of the deprecations in each, as sublists. People have been fixing them piecemeal, so I wanted to be able to track exactly what was and wasn't done. I also checked off all the ones that had been done, and added links to their respective PRs.

Also, breaking it down like this hopefully makes these smaller tasks, so more people can jump in and help with little parts.

@trek
Copy link
Member

trek commented Aug 10, 2015

I think @dstaley is working on a much larger version of this: dstaley/ember.js#1

Although I don't understand why he's tracking the issue on his branch vs mainline

@trek
Copy link
Member

trek commented Aug 10, 2015

Could the two of you work together to flesh out the full list please? Thanks.

@wifelette
Copy link
Member

I commented there. Hopefully he'll jump in here and help :)

@dstaley
Copy link
Contributor

dstaley commented Aug 10, 2015

@wifelette Definitely up for helping out! Just let me know what makes the most sense as far as where to put tracking issues, where to send PRs, etc. My original idea was to track the issues on my branches as PRs are sent to the branch, and then send on one big PR to mainline when it was done. If it makes more sense to track here and send PRs to a branch or something else just let me know.

@wifelette
Copy link
Member

Howdy @dstaley! I think my thought was mostly the usual—PRs submitted to the website repo, added as a reference here, checked off when merged :)

I can't help write much of this content, but I can keep on top of it and get the PRs merged as quickly as possible, which has been part of the issue in the past. As in, sometimes people try helping with this, but then the PRs just sit around and waste away. Innocently of course, it's just that there's SO much else happening. So if I'm aggressively getting things merged, and we can get a couple people to help, this should hopefully be a much quicker task than it previously was.

Having the main bit of work living here means it's more discoverable, so I'm hoping more people notice it, and I'll be able to draw more specific attention to it. (Also I hadn't noticed your thing when I spent the couple hours sifting through everything last night, or the job might've been a lot easier 😜 Sounds like my life 😄 )

It's probably helpful to comment here when self-assigning an item, or add a username after the list item, a.k.a. "I'm tackling X", so other folks choose different things to tackle.

@wifelette
Copy link
Member

Update: a bunch of helpful folk are hanging in the #documentation channel on the Ember Community Slack, interested in this very topic :)

duggiefresh added a commit to duggiefresh/website that referenced this issue Aug 10, 2015
- Adds links out to MDN for more information.

See emberjs#2256
duggiefresh added a commit to duggiefresh/website that referenced this issue Aug 10, 2015
- Adds links out to MDN for more information.
- Apologies for the uninventive examples.

See emberjs#2256
duggiefresh added a commit to duggiefresh/website that referenced this issue Aug 10, 2015
- Adds links out to MDN for more information.
- Apologies for the uninventive examples.

See emberjs#2256
@wifelette wifelette self-assigned this Aug 11, 2015
duggiefresh added a commit to duggiefresh/website that referenced this issue Aug 12, 2015
@wifelette
Copy link
Member

Thanks @martndemus, updated and linked to commit :)

@workmanw
Copy link
Contributor

Ember.SortableMixin seems to be gone in 2.0. I briefly looked around, but found no mention of its deprecation or disappearance.

@ultimatemonty
Copy link
Contributor

Request to add emberjs/ember.js#11493 to the deprecations guide along with the reasoning behind it and acceptable alternatives. There are only 2 events that run only on component first run (didInitAttrs and didInsertElement) and the only place to init something like a jquery plugin wrapper is didInsertElement.

Thanks for all you folks do!

@wifelette
Copy link
Member

@ultimatemonty Added to the list, thanks!

@wifelette
Copy link
Member

@workmanw SortableMixin was part of the controller feature in 1.13, and it was removed (along with many other controller deprecations). However, it's still available in the Legacy Controller Addon (at https://github.com/emberjs/ember-legacy-controllers/blob/master/addon/utils/sortable-mixin.js), and it would great to get a PR pointing people to the Mixin for that feature :)

romulomachado added a commit to romulomachado/ember-website that referenced this issue Oct 9, 2015
romulomachado added a commit to romulomachado/ember-website that referenced this issue Oct 9, 2015
romulomachado added a commit to romulomachado/ember-website that referenced this issue Oct 9, 2015
romulomachado added a commit to romulomachado/ember-website that referenced this issue Oct 9, 2015
@workmanw
Copy link
Contributor

workmanw commented Oct 9, 2015

@wifelette et al -- I'd be happy to submit a pull for SortableMixin ... but it wasn't actually deprecated in a literal sense, it was just kind of removed without warning. So it doesn't fit within the literal sense of "DEPRECATIONS ADDED IN 1.13". But if that seems okay, I'll open a PR.

@workmanw
Copy link
Contributor

workmanw commented Nov 6, 2015

@wifelette Now that Ember.SortableMixin has been deprecated in Ember 1.x. I've added #2405.

mixonic pushed a commit to mixonic/website that referenced this issue Nov 14, 2015
mixonic pushed a commit to mixonic/website that referenced this issue Nov 14, 2015
@toddjordan
Copy link
Contributor

I can start gradually working through the some of the rest of these. I'll start at the batch from 1.13.7 and start working my way back. Starting with TrackedArray and SubArray...

@toddjordan
Copy link
Contributor

ok, this week I'll move on to the @guid and @item deprecations from 1.13.2

@toddjordan
Copy link
Contributor

Moving on to the remaining 1.13.5 deprecations

@acorncom
Copy link
Contributor

acorncom commented Jan 8, 2016

@wifelette @toddjordan #2428 was just merged. I just updated the checkboxes above to reflect that, which means at this point, I believe all deprecations are documented. Is that true?

@wifelette I'll leave the pleasure of closing this issue to you :-)

@wifelette
Copy link
Member

Hah! I'm out this week and was legit sad that someone else would get to close it ;) Thanks everyone for helping make this happen!

@Gaurav0
Copy link
Contributor Author

Gaurav0 commented Jan 8, 2016

Thank you all for your hard work on this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

8 participants