Skip to content
This repository was archived by the owner on Mar 16, 2026. It is now read-only.

Relationship Values not clearing via admin API#4755

Open
millsy wants to merge 3 commits into
keystonejs:masterfrom
millsy:master
Open

Relationship Values not clearing via admin API#4755
millsy wants to merge 3 commits into
keystonejs:masterfrom
millsy:master

Conversation

@millsy
Copy link
Copy Markdown

@millsy millsy commented Jul 31, 2018

When removing a relationship value in the admin API, the value wasn't clearing because there was a check forvalue === undefined which resulted in the update ignoring that change.

Tested and confirmed to be working

@stennie stennie added the bug label Sep 25, 2018
Copy link
Copy Markdown
Contributor

@autoboxer autoboxer left a comment

Choose a reason for hiding this comment

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

@millsy, I made a small suggestion to clean up your pull request, but will get this merged soon once you're able to make the update.

Comment thread fields/types/relationship/RelationshipType.js
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants