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

Delete old deprecation initializer #2748

Conversation

calcaide
Copy link
Collaborator

Description

NOTE: notice this PR will not merge to main but to ICU-16544-upgrade-ember-data-addons-api, here the PR. The upgrade ember-data in addons/api is a dependency/necessary work to get rid of the ember-data:deprecate-array-like deprecation. Therefore, to not introduce any transitional deprecation messages within main while we upgrade ember-data.

🎟️ Jira ticket

How to Test

  • Run the UI without deprecation messages. A section of the Admin UI that will show ember-data:deprecate-array-like deprecation message will be: Host Catalogs > Host Catalog > Tab: Hosts. So navigate to that section without seeing a deprecation message.
  • Run the tests suites.

@calcaide calcaide self-assigned this Mar 25, 2025
Copy link

vercel bot commented Mar 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
boundary-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 27, 2025 4:27pm
boundary-ui-desktop ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 27, 2025 4:27pm

…ons-api' into chore-ICU-16540-clean-old-deprecation-initializers
Copy link
Collaborator

@DhariniJeeva DhariniJeeva left a comment

Choose a reason for hiding this comment

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

I tested it locally and do not see the deprecation warnings. Thanks for the cleanup!

Copy link
Collaborator

@lisbet-alvarez lisbet-alvarez left a comment

Choose a reason for hiding this comment

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

Thank u for cleaning these up!

@calcaide calcaide merged commit eb7366c into ICU-16544-upgrade-ember-data-addons-api Mar 28, 2025
11 checks passed
@calcaide calcaide deleted the chore-ICU-16540-clean-old-deprecation-initializers branch March 28, 2025 16:55
lisbet-alvarez added a commit that referenced this pull request Apr 3, 2025
* chore: 🤖 upgrade ember-data to v5.3.11

* Add missing copyright headers

* refactor: 💡 remove comment

* chore: 🤖 update yarn.lock due to merge conflict

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-16544

* fix: 🐛 undo ember-source upgrade (should be upgraded w/ ember)

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-16544

* chore: 🤖 move dep to devDeps & resolve test-helper version

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-16544

* fix: 🐛 unique record id error bug fix using store.push

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-16544

* refactor: 💡 cleanup

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-16544

* Delete old deprecation initializer (#2748)

* refactor: 💡 import from'ember-data/store'

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-16544

* refactor: 💡 clean up comments

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-16544

* refactor: 💡 remove compatWith comment

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-16544

---------

Co-authored-by: Carlos Alcaide Corvo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants