Front

[aotaduy]

Deberiamos eliminar consiole.log innecesarios

Eliminar metodos vacios como:

Creo qeu falta proteger las rutas de admin a nivel front con un guard.

https://github.com/AgostinaChiara/tpDSW/blob/9d004f9981be080324f488eac29893570d3d1de3/src/app/pages/login/login.component.ts#L24

src/app/app-routing.module.ts

normalmente los urls van en castellano para que sean faciles de leer y se usa kebab-case en vez de camelCase los nombres de las clases css lo mismo

https://github.com/AgostinaChiara/tpDSW/blob/9d004f9981be080324f488eac29893570d3d1de3/src/app/pages/book-list/book-list/book-list.component.html#L17

VEo estilos inline mezclado con clases atomicas de css de bootstrap, me parece que de usar bootstrap las clase atomicas se deben usar solo para excepciones y sino usar clases normales del componente. No me parece bien usar estilos inline en angular. es mejor usar clases en el componente o globales si s on cosas que se repiten

colores y demas deberian estar en variables de css o de scss

https://github.com/AgostinaChiara/tpDSW/blob/9d004f9981be080324f488eac29893570d3d1de3/src/app/pages/book-list/book-list/book-list.component.ts#L22

el uso de _ para vars privadas no es muy comun en js

https://github.com/AgostinaChiara/tpDSW/blob/9d004f9981be080324f488eac29893570d3d1de3/src/app/pages/book-list/book-list/book-list.component.css#L22

No es recomendable crear estilos para los tags vean https://frontstuff.io/you-need-to-stop-targeting-tags-in-css

es mejor aplicar clases a cada elemento y crear los estilos a partir de eso

Tal vez se podria refactorizar src/app/pages/crud/create-book/create-book.component.ts y src/app/pages/crud/update-book/update-book.component.ts

pare reusar ya que los forms son iguales o casi iguales

[AgostinaChiara]

Profe ahi borramos los console.logs innecesarios y metodos vacios que nos quedaron. Las rutas ya estan protegidas en el front con el guard, tenemos para que obligue a iniciar sesion, y rutas especificas para los administradores. Acomodamos tambien el tema de los estilos inline que teniamos y agregamos a los estilos globales. por ultimo tambien creamos un nuevo componente, que es un formulario que reutilizamos en el create y update del libro. Lo unico que no cambiamos fue el tema de los nombres en español y con kebab-case, ya nos quedo para la proxima, pero si es necesario cambiarlo bien lo hacemos.

[aotaduy]

No hay problema con que cambien lo de español lo puede dejar y lo de kebabCase me interesa que lo tengan a cuenta a futuro. porque justamente son cosas que hay que tener en cuenta desde el prinicpio porque son costosas de cambiar despues.

[aotaduy]

Entonces si ya esta todo cierren esto