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.

Sign up for GitHub

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

Jump to bottom

Support for HEAD method from request manager #9632

Closed
bobisjan opened this issue Jan 9, 2025 · 1 comment · Fixed by #9689
Closed

Support for HEAD method from request manager #9632

bobisjan opened this issue Jan 9, 2025 · 1 comment · Fixed by #9689
Labels
🏷️ bug This PR primarily fixes a reported issue

Comments

@bobisjan
Copy link

bobisjan commented Jan 9, 2025

The HTTPMethod allows to use HEAD, but request validation does not. Should be the validation updated to allow HEAD?
@github-project-automation github-project-automation bot moved this to needs triage in EmberData Jan 9, 2025
@runspired
Copy link
Contributor

yes, will fix thanks! Completely forgot that was a valid thing 😬

@runspired runspired added the 🏷️ bug This PR primarily fixes a reported issue label Jan 9, 2025
@runspired runspired mentioned this issue Feb 22, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🏷️ bug This PR primarily fixes a reported issue
Projects
EmberData
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: support all known http methods emberjs/data
2 participants
@bobisjan @runspired