fix: user settings PATCH treats null body values as field omission#842
fix: user settings PATCH treats null body values as field omission#842tmdeveloper007 wants to merge 2 commits into
Conversation
|
@TESTPERSONAL is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
8f42a2c to
a3ffd70
Compare
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Two issues:
-
Missing EOF newline on
e2e/landing.spec.js. -
Null allowed into boolean field: Changing guards from
typeof is_public === 'boolean'tois_public !== undefinedallows{ is_public: null }through —nullwill be assigned to a typed boolean column. Change tois_public !== undefined && is_public !== nullto properly handle field omission vs explicit null.
fix: user settings PATCH treats null body values as field omission