-
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
Salvador cuellar #1
base: salvador-cuellar-review
Are you sure you want to change the base?
Conversation
console.log(x); | ||
} | ||
|
||
let agregar = (framework) => { |
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.
Parameters should always have a type, Typescript only infers a type when a variable is declared and assigned, so if we don't specify the type on parameters, the type will be 'any'.
<input type="text" class="form-control prueba" placeholder="Framework"> | ||
</div> | ||
<div class="col-3"> | ||
<input type="text" class="form-control prueba2" placeholder="Descripcion" value=""> |
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.
Let's use more meaningful id/class names, for example 'input-description' instead of 'pruebaX'
Also, is a better practice to use ids on input forms since in most of the cases the elements on a form are unique.
} | ||
|
||
|
||
let prueba = () => { |
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.
- When a variable or method is not going to be reassigned, is a better practice to use
const
instead oflet
. - Let's use more meaningful method names, e.g.
addFramework
.
Ejercicio terminado sin searchbar