Thanks to visit codestin.com
Credit goes to github.com

Skip to content

GH-158 & GH-161: Corrected entity update validation and relationship update#162

Merged
vibhatha merged 3 commits into
LDFLK:mainfrom
zaeema-n:GH-158-update_validation
Jun 8, 2025
Merged

GH-158 & GH-161: Corrected entity update validation and relationship update#162
vibhatha merged 3 commits into
LDFLK:mainfrom
zaeema-n:GH-158-update_validation

Conversation

@zaeema-n
Copy link
Copy Markdown
Member

@zaeema-n zaeema-n commented Jun 8, 2025

This pr fixes the incorrect validation of graph entity updates to correctly check only for fields that are needed. It also fixes the issue where relationships were not being correctly updated when using the update endpoint in the update api.

Closes #158 and closes #161

@zaeema-n zaeema-n requested a review from vibhatha June 8, 2025 04:50
Copy link
Copy Markdown
Member

@vibhatha vibhatha left a comment

Choose a reason for hiding this comment

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

Overall LGTM!

@vibhatha vibhatha merged commit 74f3ae5 into LDFLK:main Jun 8, 2025
4 checks passed
@zaeema-n zaeema-n deleted the GH-158-update_validation branch June 8, 2025 09:34
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.

[BUG] Relationship update not working [BUG] Incorrect validation when updating an entity

2 participants