Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

api {} blank (pull #99) AND a patch to handle XML Parameters #101

Closed
wants to merge 3 commits into from

Conversation

brianhorakh
Copy link

I added / to the regex for @apiParameter parsing so we can document XML input output (using XPATH conventions) -- ex: Auth/UserId

This also includes my previous pull request #99 for @API {} which is a cheap hack for writing non-API method pages.

@PaulDapolito
Copy link

This looks good. Nice work.

@rottmann
Copy link
Member

Thanks. Will merge b98e99e in 0.7.0. but the type field must be set, it is not optional.

In 0.7.0 is a api-call function implemented, which need type too (it will be to complicated to check on every use if type is set). A separate include of .md files with / without navigation will be implementen in the next release too, so the empty type problem should be solved.

@rottmann rottmann closed this Aug 22, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants