Skip to content

[BREAKING] Remove ember-fetch #444

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Gaurav0
Copy link

@Gaurav0 Gaurav0 commented Jul 12, 2025

For #442

BREAKING:

  • Removes support for ember 3.28 and ember 4.4 (@ember/test-waiters 3.1.0 doesn't support)
  • Removes ember-fetch as a dependency. Apps dependent on ember-fetch may have to explicitly add it to dependencies.

Uses waitForFetch from @ember/test-waiters to replace the core functionality of ember-fetch, as explained in RFC 1065

Based on #443 , please review that PR first.

@Gaurav0 Gaurav0 force-pushed the remove_ember_fetch branch from 6054a4e to 62171d5 Compare July 12, 2025 18:00
@Gaurav0 Gaurav0 changed the title Remove ember fetch Remove ember-fetch Jul 12, 2025
@Gaurav0 Gaurav0 force-pushed the remove_ember_fetch branch 6 times, most recently from 7a967d9 to ea0690a Compare July 14, 2025 15:02
@Gaurav0 Gaurav0 force-pushed the remove_ember_fetch branch from 1a28e24 to da1a9b2 Compare July 14, 2025 15:37
@Gaurav0 Gaurav0 changed the title Remove ember-fetch [BREAKING] Remove ember-fetch Jul 14, 2025
@Gaurav0 Gaurav0 force-pushed the remove_ember_fetch branch from de0801e to 732b8aa Compare July 14, 2025 16:08
@Gaurav0 Gaurav0 mentioned this pull request Jul 14, 2025
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.

1 participant