Skip to content

Relationships #858

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: 5.x
Choose a base branch
from
Open

Relationships #858

wants to merge 7 commits into from

Conversation

yvo-niedrich
Copy link
Contributor

@yvo-niedrich yvo-niedrich commented Mar 31, 2025

Tidy Up

I've added the .idea folder to .gitignore... @27pchrisl maybe take a backup before checking out the branch.

Relationships with LOdata

Introduces Relationship operations for Eloquent Models (previously the $ref segment was ignored and invalid CREATE & DELETE operations were performed/attempted on the entire entity).

I've only added the new RelationshipInterface to the EloquentEntitySet - attempting to manipulate relationships with other types will result in 501 NOT IMPLEMENTED status codes. Support for more relationship types and entity types may follow - as I encounter and require them...

For more information see:

@27pchrisl
Copy link
Contributor

Hi @yvo-niedrich, could you restore the .idea folder? It's shared project configuration and is supposed to be committed. (https://www.jetbrains.com/help/phpstorm/creating-and-managing-projects.html)
Thanks!

@yvo-niedrich
Copy link
Contributor Author

May have had an old version of PHPStorm that modified the configs without my doing... Thanks for the docs, I have restored the .idea folder....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants