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

Assign is being referenced from @ember/polyfills. This prevents upgrade to Ember 4

Open cah-brian-gantzler opened this issue 3 years ago • 3 comments

Assign is being exported here https://github.com/san650/ember-cli-page-object/blob/master/addon-test-support/-private/helpers.js#L1. Why would we be exporting assign? This is being transpiled to below. This is preventing an upgrade to ember 4. Could we remove this import please

image

(Updated. Thought it was an import, turns out to be an export, which is even weirder)

cah-brian-gantzler avatar Apr 08 '22 18:04 cah-brian-gantzler

Ok I see that this assign is imported in many other files. Could this line be replaced with?

export { Object.assign as assign };

cah-brian-gantzler avatar Apr 08 '22 18:04 cah-brian-gantzler

Appears this is fixed in PR https://github.com/san650/ember-cli-page-object/pull/536/files but this was merged into the V2-beta branch

Since this branch hasnt been touched since Jan 30, any way we could maybe backport this fix so consumers can upgrade to Ember 4 without waiting on the V2 version of this addon?

cah-brian-gantzler avatar Apr 08 '22 18:04 cah-brian-gantzler

Sorry for delay.

Yes, I'd prefer to use the object spread approach as you referred above. Unfortunately, I doubt if it can be just cherry-picked, cause of many changes in beta happened and conflicts are possible. I think we still can do it manually though, w/o a big issue.

ro0gr avatar Apr 17 '22 14:04 ro0gr

Just released the fix as a part of https://github.com/san650/ember-cli-page-object/releases/tag/v1.17.12

ro0gr avatar Nov 27 '22 11:11 ro0gr