RedTurtle / redturtle.volto

Helper package to setup a RedTurtle's Plone site ready to work with Volto
GNU General Public License v2.0
1 stars 1 forks source link

add patch to modidify indexing of start index #87

Closed eikichi18 closed 8 months ago

eikichi18 commented 9 months ago

Modificato il comportamento per l'indicizzazione della start date degli eventi, perché altrimenti la vista dello scadenziario non funziona bene.

cekk commented 9 months ago

@mamico è per caso legato ai fix che hai fatto anche tu sugli eventi?

coveralls commented 9 months ago

Pull Request Test Coverage Report for Build 7289610596


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/redturtle/volto/monkey.py 3 9 33.33%
<!-- Total: 3 9 33.33% -->
Files with Coverage Reduction New Missed Lines %
src/redturtle/volto/monkey.py 1 47.44%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 6907137559: -0.2%
Covered Lines: 806
Relevant Lines: 1281

💛 - Coveralls
mamico commented 9 months ago

@mamico è per caso legato ai fix che hai fatto anche tu sugli eventi?

Eh, non lo so perché non è scritto qual'era il problema. Ma perché una monkey patch e non una pull request sul prodotto?

eikichi18 commented 9 months ago

Il problema è che a catalogo vengono salvate le varie start date delle ripetizioni solo se la data della ripetizione è superiore alla data di oggi. Quindi per lo scadenziario che ha bisogno di mostrare anche i vecchi eventi non va bene.

Non ho fatto una PR sul prodotto per questo, la logica iniziale non è sbagliata, solo non va bene per il nostro caso d'uso

mamico commented 8 months ago

Changelog e readme ?

eikichi18 commented 8 months ago

fatto