Conversation
Test Results85 tests 85 ✅ 0s ⏱️ Results for commit 8c30084. ♻️ This comment has been updated with latest results. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1145 +/- ##
==========================================
- Coverage 51.92% 51.90% -0.02%
==========================================
Files 639 638 -1
Lines 25196 25115 -81
==========================================
- Hits 13082 13036 -46
+ Misses 12114 12079 -35
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
c36ea80 to
8c30084
Compare
destanyol
left a comment
There was a problem hiding this comment.
👦 A mi em sembla bé, passo en Claude 🤖 aviam si veu alguna cosa
destanyol
left a comment
There was a problem hiding this comment.
Review
Canvi simple i net. La migració está ben estructurada (usa load_data_records amb els IDs específics, consistent amb el patró existent). Un punt a verificar.
🟡 Asimetria entre els canvis: D1/02 elimina el filtre completament
Dels 4 registres modificats, 3 canvien days=2 → days=1, però action_ds_rebuig_dades_autoconsum_dashboard (D1/02) elimina el filtre de data completament:
# D1/02 (rebuig dades autoconsum)
- ('date', '<=', (datetime.datetime.today()-datetime.timedelta(days=2)).strftime('%Y-%m-%d'))
# ← sense filtre de data# C/01, M/01 (amb i sense attachments)
- ('date', '<=', ... days=2 ...)
+ ('date', '<=', ... days=1 ...)Sense filtre de data, el dashboard mostrarà tots els casos D1/02 pendents d'enviament independentment de quan es van crear. És això el que demana la incidència? O hauria de ser també days=1 (o un altre valor) com els altres?
🟢 Migració correcta
La migració actualitza exactament els 4 registres modificats al XML, consistent amb el patró dels post-0001 i post-0002 existents.
🤖 Reviewed by Claude Code
És correcte, han demanat això perquè resulta que per les Ds mai s'hauria d'haver filtrat per data, ens ho van indicar malament la tarjeta original. |
Objectiu
Arreglar el número de dies tal i com demanen a la incidència
Targeta on es demana o Incidència
https://freescout.somenergia.coop/conversation/8128442?folder_id=103
Comprovacions