-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add type-definition #6
Conversation
lib/api-get.js
Outdated
class ApiGet extends API { | ||
|
||
/** | ||
* Validate the model, extracted from the parsed endpoint |
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.
🤔 Creo que falta el
* @returns {void}
Que usaste en el process
.
Y de pasó en todos los metodos que no hay un return también se podría agregar. De esa manera quedan todos parejos.
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.
Como descripción le pondría:
"Realiza validaciones antes de procesar. Settea el Modelo a usar y parsea el Endpoint. Importante, no es aconsejable sobreescribirlo"
lib/api-get.js
Outdated
/** | ||
* @typedef {Object} ApiGetError A instance of APIGetError class | ||
*/ | ||
|
||
class ApiGet extends API { |
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.
Este package quedó viejito.. aca podriamos exportarlo de una
module.exports = class ApiGet extends API {
lib/helpers/endpoint-parser.js
Outdated
* @property {String} [parents] The rest of the text string that does not have a specific function | ||
*/ | ||
|
||
/** |
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.
Esto deberia estar en el metodo no en la definición de la clase
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.
Deje varios cambios y mejoras..
Y una cosa más falta el Changelog
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.
Listorti
Requerimiento:
Se requiere agreagar estos TypeDef al package de API-Get
Historia de usuario
https://fizzmod.atlassian.net/browse/JCN-331
Subtarea
https://fizzmod.atlassian.net/browse/JCN-341