Skip to content

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

Closed
ryanduguid wants to merge 3 commits into
XeroAPI:masterfrom
ryanduguid:fix/files-folder-required-property
Closed

fix(files): correct Folder required property case#834
ryanduguid wants to merge 3 commits 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.

@ryanduguid
ryanduguid marked this pull request as draft August 12, 2026 17:49
@ryanduguid
ryanduguid marked this pull request as ready for review August 18, 2026 15:08
@ryanduguid

Copy link
Copy Markdown
Author

Closing this as part of cleaning up an unsolicited batch I opened across several Xero repositories. I will not reopen unless a maintainer asks for a single focused change.

@ryanduguid ryanduguid closed this Aug 20, 2026
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