issues
search
jorgejcabrera
/
rent-app-service
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
si recibis por parametro el localdatetime, en vez de instanciarlo acá adentro, es mas facil para testear.
#18
maurociancio
opened
2 years ago
1
capaz todo se deberia resolver adentro del borrower?
#17
maurociancio
closed
2 years ago
1
esta validacion deberia ir al dominio
#16
maurociancio
closed
2 years ago
1
capaz privado el metodo, no tiene mucho sentido que sea publico, o sí?
#15
maurociancio
closed
2 years ago
1
no deberias laburar con ids en las entidades
#14
maurociancio
opened
2 years ago
0
moving order status to order class
#13
jorgejcabrera
closed
2 years ago
0
cleaning order entity
#12
jorgejcabrera
closed
2 years ago
0
esta logica debe estar en la entidad y no en el servicio
#11
maurociancio
closed
2 years ago
1
xq no validar esto en el constructor?
#10
maurociancio
closed
2 years ago
3
no valida estado antes de hacer la transicion
#9
maurociancio
closed
2 years ago
2
podes usar nombres de metodos con espacios void "esto es un metodo"() {}
#8
maurociancio
closed
2 years ago
1
podes usar .any { ... }
#7
maurociancio
closed
2 years ago
1
mejora?
#6
maurociancio
closed
2 years ago
2
no pagina y filtra en memoria
#5
maurociancio
opened
2 years ago
3
puede ser parte de la clase item
#4
maurociancio
closed
2 years ago
1
no valida ningun estado, creo que no se puede prestar dos veces la misma cosa no?
#3
maurociancio
closed
2 years ago
7
this.price + this.assuranceCost funciona en groovy
#2
maurociancio
closed
2 years ago
1
los returns son opcionales en groovy
#1
maurociancio
closed
2 years ago
0