search

code4romania / war-support-un-acoperis

https://unacoperis.ro/ro
Mozilla Public License 2.0
9 stars 38 forks source link

Allocation history and periods #219

Closed x-dragos closed 2 years ago

x-dragos commented 2 years ago

Requirements for making a pull request

Thank you for contributing to our project!

Please fill out the template below to help the project maintainers review it as fast as possible and include your contribution to the project.

What does it fix?

Closes #152 Closes #153

Please mention the main changes this PR brings.

How has it been tested?

locally tested with various allocation flows

Please describe the tests that you ran to verify your changes.

x-dragos commented 2 years ago

fixed issue with no availability interval for host @andreiio @gheorghelupu17

x-dragos commented 2 years ago

am sa vorbesc cu colegii si vedem cum ne impartim taskurile @beniamin in functie de ce incarcare avem.

Multe din observatiile de "forma" le-am trecut cu vederea in reviewul facut de mine pentru ca initial mi s-a parut ca "este f urgent" taskul :) dar, ai perfecta dreptate in observatiile facute :) multumim.

beniamin commented 2 years ago

am sa vorbesc cu colegii si vedem cum ne impartim taskurile @beniamin in functie de ce incarcare avem.

Multe din observatiile de "forma" le-am trecut cu vederea in reviewul facut de mine pentru ca initial mi s-a parut ca "este f urgent" taskul :) dar, ai perfecta dreptate in observatiile facute :) multumim.

intr'adevar cele de forma nu sunt must si pot fi rezolvate si mai tarziu prin refactorizari.

x-dragos commented 2 years ago

Ar mai fi o mica problema pe care nu am tratat-o. Inregistrarile din allocation care exista deja. Avand in vedere ca ne asteptam ca feature-ul sa fie merged inainte de golive, nu era in carti la inceput. Cum vreti sa tratatm asta? Corectam datele in db cu date de inceput si sfarsit generice? Ar fi putin cam complicat sa facem interfata si maj ales calendarul sa afiseze situatiile pentru care nu au start si end date.