Allow label edits to round-trip the path fields - #381
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Folded into #378 instead, same author, and that PR is already approved. |
Label.to_dict()returnspath/path_ar(added in #369), and the admin edit form PUTs that payload straight back.LabelValidationModelis strict, so every label edit fails with:Adds both to the same discard list that already carries
id,order,updated_at,parent_label_idandchildren.Reported by @apodacaduron on #378.
Tests
to_dict()back and expects 200 (fails with 400 without the fix).uv run pytest: 869 passed, 4 skipped.