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
Generating api-spec with Scriptappy #508
Conversation
src/qix.js
Outdated
* @interface Mixin | ||
* @property {String|Array<String>} types String or array of strings containing the API-types that | ||
* will be mixed in. | ||
* @param {Object} [extend] Object literal containing the methods that will be extended on the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think all of these should be @property
instead of @param
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes! My bad..
src/intercept.js
Outdated
/** | ||
* This method is invoked when a request is about to be sent to QIX Engine. | ||
* @function InterceptorRequest#onFulfilled | ||
* @param {Object} session The session executing the interceptor. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might as well use Session
as type here since its' defined in session.js
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fix
src/qix.js
Outdated
* @property {String} genericType Corresponds to the qInfo.qType property on the generic object's | ||
* properties object. | ||
* @property {Object} session Contains a reference to the session that this API belongs to. | ||
* @property {String} handle Contains the handle QIX Engine assigned to the API. Used interally in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
handle is a number
src/qix.js
Outdated
* @property {String} type Contains the schema class name for this API. | ||
* @property {String} genericType Corresponds to the qInfo.qType property on the generic object's | ||
* properties object. | ||
* @property {Object} session Contains a reference to the session that this API belongs to. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Session
as type
src/qix.js
Outdated
* @param {Object} config The configuration object for the QIX session. | ||
* @returns {Session} Returns a new QIX session. | ||
* @param {Configuration} config The configuration object for the QIX session. | ||
* @returns {Object} Returns a new QIX session. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Session
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, double check the version comment please
@@ -0,0 +1,442 @@ | |||
{ | |||
"scriptappy": "1.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this version mean? Is this the API spec version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's scriptappy schema version..
No description provided.