Skip to content

docs(openapi): pin PurchaseOrderVendor schema field constraints#298

Merged
CryptoJones merged 1 commit into
masterfrom
docs/openapi-povendor-field-constraints
May 19, 2026
Merged

docs(openapi): pin PurchaseOrderVendor schema field constraints#298
CryptoJones merged 1 commit into
masterfrom
docs/openapi-povendor-field-constraints

Conversation

@CryptoJones
Copy link
Copy Markdown
Owner

Summary

Extends the field-length pinning sweep to the PurchaseOrderVendor component. The 15 string fields all gain explicit min/maxLength matching purchaseordervendor.schema.js:

  • 3 NOT-NULL columns (povName, povMailingAddress1, povMailingCity): minLength: 1, maxLength: 255
  • Other 255-cap fields: maxLength: 255
  • Zip fields: maxLength: 32
  • povPhone: maxLength: 64
  • povEMail: maxLength: 255 + format: email

Test plan

  • npm run lint && npm test — 761 passing.

Proudly Made in Nebraska. Go Big Red! 🌽 https://xkcd.com/2347/

Extends the field-length pinning sweep (Customer, Company, Worker,
BillingType, InventoryItem, Job, CustomerPayment) to the PO Vendor
component. The 15 string fields all gain explicit min/maxLength
matching purchaseordervendor.schema.js:

  - The 3 NOT-NULL columns (povName, povMailingAddress1,
    povMailingCity): minLength: 1, maxLength: 255
  - Other 255-cap fields: maxLength: 255
  - Zip fields: maxLength: 32 (matching the schema's max(32))
  - povPhone: maxLength: 64
  - povEMail: maxLength: 255 + format: email

SDK generators (openapi-typescript et al.) carry the bounds into
client-side types. No behavior change. 761 tests still pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@CryptoJones CryptoJones merged commit 45af994 into master May 19, 2026
3 checks passed
@CryptoJones CryptoJones deleted the docs/openapi-povendor-field-constraints branch May 19, 2026 16:42
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