Skip to content

Fix schema section in audit.service_plan.create event docs#1042

Merged
anniesing merged 1 commit into
cloudfoundry:masterfrom
cloudfoundry-incubator:pr-fix-create-service-plan-event-docs
Dec 21, 2017
Merged

Fix schema section in audit.service_plan.create event docs#1042
anniesing merged 1 commit into
cloudfoundry:masterfrom
cloudfoundry-incubator:pr-fix-create-service-plan-event-docs

Conversation

@jenspinney

Copy link
Copy Markdown
Contributor

In the documentation page for "List Service Plan Create Events", the schemas section of the example response is incorrect. This PR fixes it to match what the CC would actually respond with, including some sample simple schemas.

Story link: #153557902

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the master branch

  • [n/a] I have run all the unit tests using bundle exec rake

  • [n/a] I have run CF Acceptance Tests

* Old documentation for the returned event was incorrect in how the
schemas are represented in the event. Fixed and included some simple
example schemas.

[#153557902]
@cfdreddbot

Copy link
Copy Markdown

Hey jenspinney!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@cf-gitbot

Copy link
Copy Markdown

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/153748886

The labels on this github issue will be updated when the story is started.

@jenspinney

Copy link
Copy Markdown
Contributor Author

Travis CI test failure must either be a flake or a test broken on master. This is a small docs-only change.

@anniesing

Copy link
Copy Markdown

This PR looks good. Merging now.

@anniesing anniesing merged commit 23225ac into cloudfoundry:master Dec 21, 2017
@deniseyu deniseyu deleted the pr-fix-create-service-plan-event-docs branch January 23, 2018 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants