Vot 69

[codecov[bot]]

Codecov Report

Merging #18 into master will increase coverage by 0.11%. The diff coverage is 80.43%.

@@             Coverage Diff              @@
##             master      #18      +/-   ##
============================================
+ Coverage     83.23%   83.35%   +0.11%     
- Complexity      296      328      +32     
============================================
  Files            44       46       +2     
  Lines           704      793      +89     
  Branches         98      115      +17     
============================================
+ Hits            586      661      +75     
- Misses           74       82       +8     
- Partials         44       50       +6

Impacted Files	Coverage Δ	Complexity Δ
...vendingontime/backend/routes/AddMachineRouter.java	100% <100%> (ø)	4 <4> (?)
...e/backend/models/bodymodels/person/SignUpData.java	74.32% <100%> (ø)	44 <1> (ø)	:arrow_down:
.../vendingontime/backend/models/machine/Machine.java	68.88% <100%> (-0.35%)	18 <1> (ø)
...dingontime/backend/config/inject/ConfigModule.java	100% <100%> (ø)	10 <0> (ø)	:arrow_down:
...dingontime/backend/services/AddMachineService.java	100% <100%> (ø)	4 <4> (?)
...kend/models/bodymodels/machine/AddMachineData.java	66.66% <66.66%> (ø)	25 <25> (?)
...ntime/backend/models/location/MachineLocation.java	46.15% <0%> (ø)	5% <0%> (+1%)	:arrow_up:
... and 3 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update fe423be...f1675b7. Read the comment docs.

[agonper]

Esto va a doler. No sé cómo no hemos caído antes. Una máquina, tiene que pertenecer a alguien... Habrá que hablarlo entre todos, pero creo que la mejor solución es lo que comentó @Erasuithon de crear "Companies" que tengan "owner", "supervisors" y "technicians" (más adelante meterían incluso "customers" para hacer fidelización). La company se crea cuando el supervisor se registra y se le asigna como owner y uno de los supervisors. @dnchia , @franciscojink ¿qué opináis al respecto? No es mucho trabajo, porque no expondremos servicios para manejar "Companies" de momento, sólo será necesario un DAO y modificar los servicios y modelos existentes.

[msalegre]

Yo ya lo tenía en cuenta, @albertogonper. De hecho, iba a hacerlo para la historia de listar las máquinas de un supervisor (o algo así). Lo único que no recordaba lo de las empresas pero sí, me parece lo mejor. ¿Os parece bien que lo hagamos para esa historia?

[agonper]

Si quieres podemos trabajar los dos en esto. Creo subtareas adicionales en el JIRA para dividir mejor el trabajo?

[msalegre]

Adelante. Avísame cuando estén listas y me las asigno.