Skip to content

Footfall & Contribution Details: CSV imports, cancellations and other payment methods (main) - #1837

Merged
tarunnjoshi merged 1 commit into
mainfrom
issue-392-camp-contribution-totals-main
Oct 19, 2026
Merged

tarunnjoshi merged 1 commit into
mainfrom
issue-392-camp-contribution-totals-main

Conversation

@tarunnjoshi

Copy link
Copy Markdown
Member

Production release of ColoredCow/goonj#1823 (merged to dev on 14-Oct, tested on staging).

Refs ColoredCow/goonj-crm#392

What's in it

The same 3 files as on dev, byte for byte. These are the only differences between main and dev for these files.

  • Camps (CollectionBaseService.php): Footfall & Contribution Details totals and contributor counts now include CSV-imported contributions and come down when a contribution is cancelled or refunded. Anything paid by a method other than Razorpay or Cash/Cheque goes into the new Total Other Monetary Contributions field.
  • Events (GoonjInitiatedEventsService.php):
    • Online and cash totals are recalculated from all Completed contributions instead of being added to on every save.
    • CSV imports and cancellations are handled the same way as camps, plus the same "Other" total.
    • The material form now counts the newest material donor.
  • New script cli/backfill-collection-source-contribution-stats.php corrects existing camp records. It is a dry run by default, and APPLY=1 saves.

Before merging (manual, on prod)

  1. Create Total Other Monetary Contributions (Alphanumeric, Text) in the Footfall & Contribution Details custom group.
  2. Create Total Other Monetary Contributions (Number, Text) in the Goonj Events Outcome custom group.

Both must end up named Total_Other_Monetary_Contributions. Without them the totals update fails.

After merging

  1. Clear the CiviCRM cache.
  2. Check that the prod worker is on the same commit.
  3. Run the backfill dry run, review the report, then run it with APPLY=1.

🤖 Generated with Claude Code

… other payment methods (main)

Brings the camp and event contribution-total changes from dev (PR #1823)
to main:

- Camp and event totals and contributor counts now include CSV-imported
  contributions and come down when a contribution is cancelled or
  refunded; only Completed contributions are counted.
- New "Total Other Monetary Contributions" total for camps and events:
  any payment method that is not Razorpay or Cash/Cheque.
- Event online/cash totals are recalculated from all Completed
  contributions instead of being added to on every save.
- Event material footfall counts the newest material donor.
- New cli/backfill-collection-source-contribution-stats.php to correct
  existing camp records (dry run by default).

Refs coloredcow-admin/goonj-crm#392

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 19, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 90c89d24-e4cc-49d1-8e5f-6149e2901d0b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tarunnjoshi
tarunnjoshi merged commit eaae2d4 into main Oct 19, 2026
3 checks passed
@tarunnjoshi
tarunnjoshi deleted the issue-392-camp-contribution-totals-main branch October 19, 2026 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant