Skip to content

Removing duplicate fields #661

@owenbendavies

Description

@owenbendavies

Currently if an API endpoint only has a desc (which is fairly common), grape-swagger uses that description for the summary, description, and response description. To me, that seems like duplication, and better would be to not set the summary unless it is specifically defined

Do people agree / disagree and why?

If it was agreed, I would be happy to do the PR, but wanted to check before putting in the effort.

Activity

LeFnord

LeFnord commented on Feb 8, 2018

@LeFnord
Member

please have a look here #629
it seems, there is actual no progress, so feel free to take over the PR

owenbendavies

owenbendavies commented on Feb 8, 2018

@owenbendavies
Author

Related, but I was referring to the extra fields under paths. Would also be nice to introduce a documentation method to entities as well to be used. In which case I'll make a PR for this first, then have a look at that PR too.

added a commit that references this issue on Mar 17, 2018
3057a2c
added a commit that references this issue on Feb 9, 2019
d586a1c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @LeFnord@owenbendavies

        Issue actions

          Removing duplicate fields · Issue #661 · ruby-grape/grape-swagger