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

Give credit for JSON schema generation to Drafter #123

Closed
wants to merge 1 commit into from

Conversation

alexturek
Copy link

No description provided.

@alexturek
Copy link
Author

Hey Daniel, been a while!

I spent a couple hours going nuts trying to figure out how JSON schema (that I didn't write) was turning up in the Aglio documentation, and then I learned about Drafter...

Although now that I look in package.json I don't see it anywhere. How are those schemas getting into the docs?? (feel free to close this PR if it's not accurate)

@danielgtaylor
Copy link
Owner

@alexturek hey! Hope everything is going well. We should catch up some time!

How did you install Aglio? Are you using the new betas (via npm install aglio@beta) or the latest stable/normal release? The betas should be using Drafter while the normal release hasn't been updated to use it yet. If you're using the beta then I'll go ahead and update the README in the theme-engine branch 👍

@alexturek
Copy link
Author

Oh haha I should have checked the version, I didn't think the guy who set
it up for us was using a beta

Our version is "aglio": "^2.0.0-beta1",
On Jun 17, 2015 8:35 PM, "Daniel G. Taylor" notifications@github.com
wrote:

@alexturek https://github.com/alexturek hey! Hope everything is going
well. We should catch up some time!

How did you install Aglio? Are you using the new betas (via npm install
aglio@beta) or the latest stable/normal release? The betas should be
using Drafter while the normal release hasn't been updated to use it yet.
If you're using the beta then I'll go ahead and update the README in the
theme-engine branch [image: 👍]


Reply to this email directly or view it on GitHub
#123 (comment).

@danielgtaylor
Copy link
Owner

@alexturek this is now up to date on the main readme after merging the theme engine branch onto master. Thanks!

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.

None yet

2 participants