Transferred-in reg still appears under the roster "unpaid" payment filter
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
app/models/event_registration.rb で paid_in_full、not_paid_in_full、payment_status、および既存の not_transferred_in scopes を読むところから始めます。それらの動作をダッシュボードの課金対象の基準および EventRevenueFigures と比較し、続いて roster の Paid/Unpaid 件数と payment_status フィルターを確認します。transferred-in 登録が一貫して除外され、roster がダッシュボードの合計と一致すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Follow-up from #1944 / PR #2044.
A transferred-in registration's balance lives on its source registration, so it owes nothing to the new event. Reporting totals already exclude it (dashboard billable basis + EventRevenueFigures), but the roster payment-status filter does not: EventRegistration.not_paid_in_full (raw SQL summing allocations vs events.cost_cents) still matches a transferred-in reg (0 allocations, cost > 0), so it shows up under payment_status: "unpaid".
Suggested fix
- Exclude transferred-in regs from
not_paid_in_full(and treat them consistently inpaid_in_full/ thepayment_statusscope), e.g.AND transferred_from_registration_id IS NULL, mirroringnot_transferred_in. - Confirm the roster "Paid/Unpaid" counts and the
payment_statusfilter agree with the dashboard's billable basis.
Context
app/models/event_registration.rb—paid_in_full/not_paid_in_full/payment_statusscopes;not_transferred_inscope already exists.- Deliberately scoped out of #1944 (a filter, not a reporting total).
- 主要言語
- Ruby
- スター
- 15
- フォーク
- 28
- 平均マージ
- 1日 4時間
- マージ済み PR（30日）
- 227
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
rubyforgood/awbw のほかの issue
-
blocked
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
rubyforgood/awbw#2824 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
rubyforgood/awbw#2822 ·
メンテナーはふだん 1 日以内に返信
-
Update callout faq defualt text when the new person subscription launches対応中かも @maebeale が 77 日前に担当しました。 オープンblocked
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
rubyforgood/awbw#2095 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
rubyforgood/awbw#2088 ·
メンテナーはふだん 1 日以内に返信
-
Published story edit: paragraph spacing too large by default, hard to read再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープンenhancement stakeholder-request stories
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
rubyforgood/awbw#1521 ·
メンテナーはふだん 1 日以内に返信
rubyforgood/awbw の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
up-for-grabs/up-for-grabs.net#6330 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 72/100
TheOdinProject/curriculum#31490 ·
メンテナーはふだん 1 日以内に返信
-
Documentation: RSpecRails/FileFixture and RSpecRails/ReceivePerformLater cops missing from websiteオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
rubocop/rubocop-rspec_rails#136 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
hotwired/turbo-rails#791 ·