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

[ENG-2490] Upgrade ember to v3.26 #1203

Merged
merged 96 commits into from
Jun 2, 2021
Merged

[ENG-2490] Upgrade ember to v3.26 #1203

merged 96 commits into from
Jun 2, 2021

Conversation

fabmiz
Copy link
Member

@fabmiz fabmiz commented May 6, 2021

Purpose

Upgrade core ember-* to v3.26 (latest stable) + required dependency upgrades

Developer experience

App user experience

  • some speedup with most (heavily used) upgrade to modern ember (glimmer and tracked properties)

Summary of Changes

Changes fall into these categories:

  1. Dependency upgrades and addressing breaking changes
  2. Async tasks
  3. ember-data -> @ember-data
  4. OsfButton -> BsButton
  5. nodeLicense validations are now nested

Upgraded outside ember-cli-update

Upgraded via ember-cli-update

New addons

ember-concurrency-async (support for async tasks)
ember-concurrency-ts (helpers for async tasks)

eslint plugins: eslint-plugin-import, eslint-plugin-node, eslint-plugin-prettier, eslint-plugin-ember-concurrency

Removed addons

ember-youtube (unused and outdated)
ember-cli-uglify (replaced by ember-cli-terser)
tslint (dropped in favor of typescript-eslint)
tslint-consistent-codestyle (dropped bc it's tslint plugin)
tslint-eslint-rules (dropped bc it's tslint plugin)
ember-oo-modifiers (replaced by ember-modifier)
ember-concurrency-test-waiter (usage replaced by @ember/test-helpers waitFor)
ember-concurrency-decorators (decorators are now part of ember-concurrency v2)

Side Effects

QA Notes

fabmiz and others added 29 commits May 7, 2021 08:45
Notable Changes
Polyfills have been removed entirely and with it the need to bundle the pluralization rules as they now come from the native Intl.PluralRules API
Intl.MessageFormat parser and compiler updated which changes how text and tags are escaped
Breaking Changes
Node 8 support dropped
Intl.RelativeTime polyfill has been replaced with the native API which behaves entirely different than the previous older spec implementation. Read more in the Migration Document
Escaping text has changed and HTML tags now need to be escaped. Read how in the Migration Document
Removes shortNumber formatting in favor of newly added native implementation using the "notation": "compact" parameter for Intl.NumberFormat i.e.,
Replace usage with <BsButton>
Why: subclassing ember-bootstrap component classes no longer supported
ember-bootstrap/ember-bootstrap#1104
@fabmiz fabmiz requested a review from aaxelb May 19, 2021 14:47
Copy link
Contributor

@aaxelb aaxelb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome work! i've looked at 200/831 files so far with only minor comments -- will circle back and skim the rest....

.eslintrc.js Outdated Show resolved Hide resolved
app/adapters/osf-adapter.ts Show resolved Hide resolved
app/adapters/osf-adapter.ts Show resolved Hide resolved
app/adapters/collection.ts Show resolved Hide resolved
app/adapters/user-registration.ts Show resolved Hide resolved
app/services/current-user.ts Show resolved Hide resolved
app/transforms/node-license.ts Outdated Show resolved Hide resolved
app/utils/capture-exception.ts Show resolved Hide resolved
app/utils/default-to.ts Show resolved Hide resolved
aaxelb
aaxelb previously approved these changes May 20, 2021
Copy link
Contributor

@aaxelb aaxelb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

outstanding ✨ ⛰️

had more comments, but nothing major

i already feel a whole lot better about ember-osf-web being held up as the big example for ember with typescript

config/deprecation-workflow.js Show resolved Hide resolved
lib/osf-components/addon/components/file-icon/component.ts Outdated Show resolved Hide resolved
node-tests/fixtures/blueprints/osf-model/model.ts Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
tests/acceptance/dashboard-test.ts Outdated Show resolved Hide resolved
tests/engines/registries/acceptance/draft/draft-test.ts Outdated Show resolved Hide resolved
tests/unit/services/ready-test.ts Outdated Show resolved Hide resolved
aaxelb
aaxelb previously approved these changes May 26, 2021
Copy link
Contributor

@aaxelb aaxelb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍡 🎼

Fix noteworthy-and-popular-project tooltip
@fabmiz fabmiz merged commit 59a18ae into develop Jun 2, 2021
@fabmiz fabmiz added this to the 21.6.0 milestone Jun 28, 2021
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.

None yet

3 participants