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.

Sign up for GitHub

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

Jump to bottom

add in property to parameters #148

Merged
merged 1 commit into from
Feb 12, 2024
Merged

add in property to parameters #148

merged 1 commit into from
Feb 12, 2024

Conversation

frankkilcommins
Copy link
Collaborator

No description provided.

@frankkilcommins frankkilcommins added documentation Improvements or additions to documentation implementor-draft In scope for first version labels Feb 9, 2024
@frankkilcommins frankkilcommins merged commit 20ce40c into main Feb 12, 2024
2 checks passed
@gcatanese
Copy link

@frankkilcommins @ndenny Is this change correct? The attribute in has been added, but type and format are not Parameter attributes.

  parameters:
    page:
      in: query
      type: integer
      format: int32

@frankkilcommins
Copy link
Collaborator Author

@gcatanese true they should not be specified. I missed that when cleaning up. There's new issue out there to fix that

@frankkilcommins frankkilcommins deleted the fix-parameter-examples branch July 31, 2024 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ndenny ndenny ndenny approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation implementor-draft In scope for first version
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@frankkilcommins @gcatanese @ndenny