ember-cli icon indicating copy to clipboard operation
ember-cli copied to clipboard

Remove DT Types

Open NullVoxPopuli opened this issue 1 year ago • 5 comments

ember-data has real types now.

Removing DT types enables downstream blueprints to also remove DT types (like the v2 addon blueprint https://github.com/embroider-build/addon-blueprint )

Removing the DT types also gives us a tooon of fixes to the types and broader compatibility with libraries released in the last couple years :tada: :muscle:

NullVoxPopuli avatar May 09 '24 22:05 NullVoxPopuli

@NullVoxPopuli What's the blocker for this one? Does it just need more work or was it discussed somewhere and put on hold?

Windvis avatar Sep 24 '24 16:09 Windvis

no blocker -- just needs someone to finish

NullVoxPopuli avatar Sep 24 '24 16:09 NullVoxPopuli

@NullVoxPopuli I'll try to find some time to work on this :+1:

One thing that concerns me a bit is that the official EmberData types are quite different from the DT ones. If we update the setup in the blueprints then apps will automatically switch to the new setup when they update using ember-cli-update, which can potentially cause a lot of new type errors.

I know it's a blueprint and people are free to choose what changes they accept when updating but I do wonder if we can make this easier to deal with somehow.

I did the DT -> official data types conversion manually in a smallish app and it was still quite a bit of work.

I fear apps might be stuck on older blueprint versions until they switch to the new types manually before updating.

Windvis avatar Sep 25 '24 07:09 Windvis

The DT types are more than wrong, and i think we shouldn't try to support them <3

They were a stopgap.

Stuck

They'll probably also be stuck on older ember-data, which means blueprint updates are less relevant to them 🎉

NullVoxPopuli avatar Sep 25 '24 12:09 NullVoxPopuli

(I think this one can be closed now, since #10506 was merged)

Windvis avatar Oct 05 '24 17:10 Windvis