ember.js icon indicating copy to clipboard operation
ember.js copied to clipboard

Inplement runtime portion of RFC #931

Open wycats opened this issue 1 year ago • 2 comments

wycats avatar Oct 10 '24 22:10 wycats

Safari 17.6 passes locally for me. I see that browserstack is using Safari 12 (we did for 5.x). We don't actually support that.

According to: https://browsersl.ist/#q=%3E+0.25%25+and+not+dead per: https://emberjs.com/browser-support/

I think we can bump to Safari 15.6

PR: https://github.com/emberjs/ember.js/pull/20779

NullVoxPopuli avatar Oct 18 '24 17:10 NullVoxPopuli

I tried this out in an embroider app here: https://github.com/NullVoxPopuli/ember-source-pr-20776-testing

and it didn't work (seems like nothing is rendered?).

maybe whatever is going on with CI will be resolved when this reproduction is resolved?

NullVoxPopuli avatar Oct 18 '24 23:10 NullVoxPopuli

Looks like there are two copies of @glimmer/manager now, which breaks setComponentTemplate.

Supposedly, I have sourcemaps turned off image

NullVoxPopuli avatar Oct 21 '24 15:10 NullVoxPopuli

Your repro link is broken and this is not enough information to say what's going wrong. There are known module-duplication bugs that effect ember-source that are fixed only in the very latest main of embroider.

ef4 avatar Oct 21 '24 15:10 ef4

Your repro link is broken and this is not enough information to say what's going wrong.

I don't understand, the link works. Also, all templates are just.. not rendered (which I said 🤔 ).

Unless you mean something more specific?

There are known module-duplication bugs that effect ember-source that are fixed only in the very latest main of embroider.

in any case, I suppose for this feature we don't want to fix for embroider3? which means we have to disable/skip the failing test?

NullVoxPopuli avatar Oct 21 '24 15:10 NullVoxPopuli

I confirm things work as expected in the vite blueprint:

image code here: https://github.com/NullVoxPopuli/ember-source-pr-20776-testing-vite

What not-working looks like in these tests image code here: https://github.com/NullVoxPopuli/ember-source-pr-20776-testing

NullVoxPopuli avatar Oct 21 '24 16:10 NullVoxPopuli

I merged https://github.com/emberjs/ember.js/pull/20782 so this needs conflict fixes.

ef4 avatar Oct 23 '24 14:10 ef4