-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[ui] Helios and Power Select upgrades #22328
Conversation
philrenaud
commented
May 24, 2024
- Upgrading Helios to 4.3.0
- ember-element-helper was previously pinned because of an old version of ember-power-select, which is now also upgraded, so pin removed.
Ember Test Audit comparison
|
@@ -106,7 +106,7 @@ | |||
"ember-on-resize-modifier": "^1.0.0", | |||
"ember-overridable-computed": "^1.0.0", | |||
"ember-page-title": "^7.0.0", | |||
"ember-power-select": "^4.1.7", | |||
"ember-power-select": "^7.2.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: I could upgrade this to v8+, but that requires adding extra template code per https://ember-power-select.com/docs/migrate-7-0-to-8-0 and I think my time would be better suited moving away from ember-power-select raw usage generally (we use it for our global search, among other things). Helios' dropdown component will be a more than sufficient replacement for it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Checked for the changes introduced in 4.x and the only relevant ones are the contextual component renames indeed
980f46d
to
2e3e6a4
Compare
* Helios and Power Select upgrades * Renamed namespaced contextual components
* Helios and Power Select upgrades * Renamed namespaced contextual components