Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

fix(admin): purchased_by_day is gross, the PR B plan says net

Aperta
  • Qualcosa non va?
#583 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Da chiarire
Stato di attività
Attiva
Stack tecnologico
python
Ambito
backend, data-visualization

Direzione di ricerca

Read value_queries.py and admin-overview.js, then compare the behavior with Task 4 in docs/superpowers/plans/2026-09-15-admin-layer-redesign-prB-read-paths-delete.md. The issue requires a decision between gross and net totals, including implications for zero/negative days and chart range; done means the chosen behavior is consistent across aggregation, constraints, and display.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

purchased_by_day (merged in #564) is gross purchases per UTC day; the PR B plan specifies net (purchases − refunds), and says the negative days are intended ("the honest revenue line").

  • Shipped: value_queries.py _ledger_days sums row.purchased_micro only and drops days with amount ≤ 0; DailyMicroTotals.amount_micro is ge=0.
  • Plan: docs/superpowers/plans/2026-09-15-admin-layer-redesign-prB-read-paths-delete.md Task 4, purchased_by_day = purchased_micro - refunded_micro, may be negative.

Why it shipped gross: it matches the Purchased headline it sits under (aggregate_commercial_ledger reports purchased and refunded separately), and the combined Credits chart in admin-overview.js only draws a 0-based axis, so a negative day can't render.

Decision needed: keep gross (and amend the plan), or switch to net — which also means dropping ge=0, keeping ≤ 0 days, and teaching the chart a negative range. LedgerDayTotal already carries refunded_micro, so either is a small change.

Lingua principale
Python
Stelle
767
Fork
156
Merge medio
22h 2m
PR unite (30g)
88

Preparare l'ambiente

  • Include un Dockerfile o un file Docker Compose
  • Nessun modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Open-Finance-Lab/AgenticTrading

Tutte le issue di Open-Finance-Lab/AgenticTrading

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.