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

Allow more lax usage #108

Open
sazzer opened this issue Dec 2, 2024 · 1 comment
Open

Allow more lax usage #108

sazzer opened this issue Dec 2, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@sazzer
Copy link

sazzer commented Dec 2, 2024

I've just come across this and it's saved me writing it all out myself :)

However, there are two places where the types in this repository are more prescriptive than in the latest HAL spec:

  • _links - the spec lists this as OPTIONAL, but the types in this repository have it as required.
  • _links.self - the spec gives no mandate that a self link must be present, but the types in this repository are forcing it to be there.

The first of these means it's impossible to return a document that has _embedded or _templates without also including _links, and the second means it's impossible to return a document without a self link - e.g. the ephemeral response to a POST request might not have a self link but might still want to return other HAL-shaped details.

@evert
Copy link
Collaborator

evert commented Dec 2, 2024

Sounds reasonable!

@evert evert added the bug Something isn't working label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants