Skip to content

fix(files): correct Folder required property case - #834

Open
ryanduguid wants to merge 1 commit into
XeroAPI:masterfrom
ryanduguid:fix/files-folder-required-property
Open

fix(files): correct Folder required property case#834
ryanduguid wants to merge 1 commit into
XeroAPI:masterfrom
ryanduguid:fix/files-folder-required-property

Conversation

@ryanduguid

Copy link
Copy Markdown

Summary

  • correct the case of the Files Folder schema's required property from name to Name
  • retain the intended requirement while matching the schema property and documented create/update request bodies

Verification

  • parsed the OpenAPI YAML with PyYAML
  • validated { "Name": "My Docs" } against Folder with jsonschema
  • confirmed an empty Folder still fails with Name required

The documented request body fails on master because the schema requires a nonexistent lower-case name field.

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.

1 participant