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.

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

KAZOO-5248: keep doc schemas in sync with ref/ #3021

Merged
merged 8 commits into from
Dec 21, 2016
Merged

Conversation

fenollp
Copy link
Contributor

@fenollp fenollp commented Dec 20, 2016

Follow up of #3008

BTW @jamesaimonetti the docs site needs Kazoo's tag in the URL.

@fenollp
Copy link
Contributor Author

fenollp commented Dec 20, 2016

These fail because I couldn't find which file corresponds to the ref/ version:

Doc does not exist, skipping applications/crossbar/doc/acls.md
Doc does not exist, skipping applications/crossbar/doc/cccps.md
Doc does not exist, skipping applications/crossbar/doc/clicktocall.md
Doc does not exist, skipping applications/crossbar/doc/connectivity.md
Doc does not exist, skipping applications/crossbar/doc/dialplans.md
Doc does not exist, skipping applications/crossbar/doc/shared_auth.md
Doc does not exist, skipping applications/crossbar/doc/sms.md
Doc does not exist, skipping applications/crossbar/doc/temporal_rules.md

@fenollp
Copy link
Contributor Author

fenollp commented Dec 20, 2016

Also, lots of files in ref/ do not have a public version in doc/

return found[0]

def public_doc(ref_path):
doc_root = os.path.split(os.path.split(ref_path)[0])[0]
Copy link
Member

Choose a reason for hiding this comment

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

why not os.path.dirname(path) ?

@k-anderson k-anderson merged commit a5db2f7 into master Dec 21, 2016
@k-anderson k-anderson deleted the KAZOO-5248 branch December 21, 2016 18:35
jamhed pushed a commit to jamhed/kazoo that referenced this pull request Dec 23, 2016
* KAZOO-5248: segregate schemas in ref docs

* KAZOO-5248: add missing schemas

* KAZOO-5248: fix assoc

* KAZOO-5248: add missing schema

* KAZOO-5248: fix spacing in faxes.md

* KAZOO-5248: amend `make apis`

* KAZOO-5248: make apis

* KAZOO-5248: use os.path.dirname/basename
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