Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[tmdgen][open-api] Régression required pour un body avec plusieurs champs #421

Open
DricomDragon opened this issue Dec 10, 2024 · 1 comment

Comments

@DricomDragon
Copy link
Contributor

Cette régression fait suite au test de la correction de la #417.

Spec openApi :

      requestBody:
        required: true
        content:
          application/json:
            schema:
              type: object
              properties:
                businessOptIn:
                  type: boolean
                  example: true
                age:
                  type: integer
                  example: 15
                customer:
                  type: object
                  $ref: "#/components/schemas/CustomerParams"

Constaté : il manque required: true.

endpoint:
  name: Post
  method: POST
  route: customers/save
  description: Non documenté
  preservePropertyCasing: true
  params:
    - composition: Body
      name: body
      comment: Non documenté

Attendu :

endpoint:
  name: Post
  method: POST
  route: customers/save
  description: Non documenté
  preservePropertyCasing: true
  params:
  - composition: Body
    name: body
    required: true
    comment: Non documenté
@JabX
Copy link
Member

JabX commented Dec 20, 2024

Une composition est required: true par défaut, à l'inverse des autres types de propriétés. Cela ne devrait du coup pas avoir d'impact de l'ajouter.

(J'ai l'impression que c'est pas marqué dans la doc par contre ?)

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

No branches or pull requests

2 participants