Skip to content

Conversation

@pgcarter
Copy link
Contributor

@pgcarter pgcarter commented Jan 8, 2019

Open API plugin will accept optional tagDescriptionsPropertiesFile attribute.
The attribute expects a yaml file containing tag name and description

Implements issue #65

Open API plugin will accept optional tagDescriptionsPropertiesFile attribute.
The attribute expects a yaml file containing tag name and description

Implements issue ePages-de#65
@coveralls
Copy link

coveralls commented Jan 8, 2019

Coverage Status

Coverage increased (+0.06%) to 94.603% when pulling 39ad989 on pgcarter:issue-65-tags-properties-file into a3a6230 on ePages-de:master.

Copy link
Contributor

@mduesterhoeft mduesterhoeft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks also for this contribution. Looks really good to me! Could you please apply the naming change I suggested.

servers = servers,
title = title,
description = apiDescription,
tags = tags,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we call this tagDescriptions please (everywhere you now use tags)?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No worries will make that change

@mduesterhoeft mduesterhoeft merged commit 875cc8d into ePages-de:master Jan 11, 2019
@mduesterhoeft
Copy link
Contributor

@pgcarter I added the two new capabilities you added to the sample application as well.

And the result using https://editor.swagger.io/ is really nice - thanks again.

image

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.

3 participants