Confirm: comped CE fees count as org subsidy in the revenue report
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
Direzione di ricerca
Esamina app/services/event_revenue_figures.rb e l'esempio di tariffe CE scontate in spec/services/event_revenue_report_spec.rb, quindi confronta la gestione esistente degli sconti sulle registrazioni con il flusso di allocazione CE in app/views/continuing_education_registrations/_payment_history.html.erb. Conferma il trattamento contabile degli sconti CE completi e parziali e se la pagina dei ricavi necessita di righe di sconto separate; il lavoro è completato quando la policy è concordata e il comportamento del report e i test sono conformi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The problem
A CE fee can only ever end up in one of three places: paid, comped (a discount allocation), or owed. #2072 made "paid" and "owed" real figures on the events revenue report — but nothing claimed the comped slice, so a fully-discounted CE fee was neither collected nor outstanding and disappeared from the report entirely.
Example — a $60 CE fee comped in full:
| Before #2072 | #2072 as first written | #2072 as merged | |
|---|---|---|---|
| Fees collected | $0 | $0 | $0 |
| Outstanding | $60 (never collectable) | $0 | $0 |
| Org subsidy | $0 | $0 | $60 |
| Net | $0 | $0 | −$60 |
The middle column is the hole: $60 of value the org gave away, invisible on every figure.
What we did
Counted CE discounts as org subsidy, matching how a discounted registration fee is already treated. Net and total expected drop by the comped amount. Implemented in EventRevenueFigures (app/services/event_revenue_figures.rb) — it's the one place the batched report figures deliberately diverge from EventDashboard.
What needs confirming
- Is "the org gave away $60" the right read of a comped CE fee for annual-report purposes? (The alternative is treating CE comps as never-was-revenue, i.e. leaving them out of every bucket — which is what was happening by accident.)
- Should the same reasoning extend to a partially comped CE fee ($60 fee, $20 discount, $40 paid → $20 subsidy)? That's what the code does today; flagging it because it wasn't discussed separately.
- Does the revenue page need to show CE discounts broken out from registration discounts, or is one combined "Discounts" line enough?
Context
- Introduced by #2072
app/services/event_revenue_figures.rb—discount_centssums Discount allocations across bothEventRegistrationandContinuingEducationRegistrationspec/services/event_revenue_report_spec.rb— "discounted CE fees" describes the expected accounting- CE comps are reachable in the UI today: the allocations page for a CE registration (
app/views/continuing_education_registrations/_payment_history.html.erb) has an "Add discount" action
- Lingua principale
- Ruby
- Stelle
- 15
- Fork
- 28
- Merge medio
- 1g 12m
- PR unite (30g)
- 262
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di rubyforgood/awbw
-
blocked
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
rubyforgood/awbw#2824 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
rubyforgood/awbw#2822 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
rubyforgood/awbw#2160 ·
I maintainer di solito rispondono entro 1 giorno
-
Update callout faq defualt text when the new person subscription launchesForse già presa @maebeale l’ha presa 78 giorni fa. Apertablocked
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
rubyforgood/awbw#2095 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
rubyforgood/awbw#2088 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di rubyforgood/awbw
Issue simili
-
triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
rubyaustralia/ruby_au#497 ·
-
ConcurrencyClaim sends NOTIFY even when enable_listen_notify is falseForse già presa @eidam l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
bensheldon/good_job#1891 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
TheOdinProject/curriculum#31493 ·
I maintainer di solito rispondono entro 1 giorno