Skip to content

docs: add donation readiness checklist before enabling donations#46

Merged
MrImmortal09 merged 2 commits into
iiitl:mainfrom
Abhishek-Dige:docs/donation-readiness-tracker
Apr 12, 2026
Merged

docs: add donation readiness checklist before enabling donations#46
MrImmortal09 merged 2 commits into
iiitl:mainfrom
Abhishek-Dige:docs/donation-readiness-tracker

Conversation

@Abhishek-Dige

@Abhishek-Dige Abhishek-Dige commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

Description

Documents the legal, financial, and operational requirements that must be fulfilled before IIITL / Axios can accept donations through the /giving page. This includes banking setup, 80G tax registration, audit trails, governance policies, technical infrastructure, and donor communication processes.

No code changes are included in this PR. The /giving page remains as "Coming soon" placeholder until all critical requirements are met.

Closes #23


Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactor / code style

Checklist

  • I have read CONTRIBUTING.md
  • My branch follows the naming convention (feat/, fix/, docs/, etc.)
  • My commits follow the Conventional Commits format
  • I have run pnpm lint and there are no errors
  • I have tested my changes locally (pnpm dev)
  • I have updated documentation where necessary
  • New and existing UI uses design-system tokens (no arbitrary hex values)
  • I have linked the related issue above

Documentation Overview

The new docs/giving-readiness.md covers:

Critical Requirements (Must Complete Before Launch)

  1. ✅ Dedicated bank account for donations
  2. ✅ 80G tax exemption registration
  3. ✅ Accounting system setup
  4. ✅ Payment gateway integration
  5. ✅ Governance policies (board-approved)
  6. ✅ Receipt generation system

Complete Coverage Areas

  • Banking Infrastructure: Payment gateways, transaction controls, dual authorization
  • Legal Compliance: 80G/12A registration, FCRA (if needed), CSR-1
  • Financial Reporting: Accounting systems, annual audits, statutory filings
  • Governance: Internal policies, board oversight, transparency reporting
  • Technical Requirements: Secure payments, donor database, reconciliation
  • Communication: Donor relations, impact reporting, acknowledgment processes

Next Steps

The issue will remain open as a tracker. Progress can be tracked by checking off items in the markdown document as they are completed.


Screenshots / Screen Recording

Before After
N/A (docs only) N/A (docs only)

Summary by CodeRabbit

  • Documentation
    • Added a comprehensive pre-launch checklist for enabling donations via the /giving page: covers banking/payment setup, legal/regulatory requirements, financial reporting and audit readiness, governance and transparency, technical/operational site requirements, donor communications and workflows, phased implementation timeline, responsible parties tables, risk assessment and mitigations, minimum launch criteria, and revision history.

@vercel

vercel Bot commented Apr 12, 2026

Copy link
Copy Markdown

@Abhishek-Dige is attempting to deploy a commit to the mrimmortal09's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Apr 12, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8f6893aa-f1a3-4bfa-ab71-35ef6762200d

📥 Commits

Reviewing files that changed from the base of the PR and between 09b4ae2 and 3961ede.

📒 Files selected for processing (1)
  • docs/giving-readiness.md
✅ Files skipped from review due to trivial changes (1)
  • docs/giving-readiness.md

Walkthrough

Added docs/giving-readiness.md, a comprehensive pre-launch checklist documenting requirements for enabling donations via the /giving page: banking setup, legal/regulatory compliance (80G/12A/etc.), financial reporting and audit readiness, governance, technical/payment integration, communication workflows, timeline, and risk assessment.

Changes

Cohort / File(s) Summary
Donation Readiness Documentation
docs/giving-readiness.md
New 371-line document providing a pre-launch checklist for the /giving page: banking and payment gateway requirements, 80G/12A/FCRA and other registrations, bookkeeping and audit readiness, governance and donor policies, technical/payment/webhook requirements, communication workflows, phased timeline, responsible parties, risk assessment, and acceptance criteria.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding a donation readiness checklist documentation before donations are enabled.
Linked Issues check ✅ Passed The PR fully satisfies issue #23's requirements: documentation of prerequisites (banking, 80G, accounting, governance) in docs/giving-readiness.md, and no code changes for payment wiring.
Out of Scope Changes check ✅ Passed All changes are in-scope: only a new documentation file addressing the linked issue's requirements; no code changes or unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 3

🧹 Nitpick comments (1)
docs/giving-readiness.md (1)

365-369: Use unambiguous date format in revision metadata.

12-04-2026 is ambiguous (DD-MM vs MM-DD). Prefer ISO YYYY-MM-DD.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docs/giving-readiness.md` around lines 365 - 369, Replace the ambiguous date
string "12-04-2026" with the ISO format "2026-04-12" in the document; update
both occurrences shown in the diff (the revision table entry containing
"12-04-2026" and the "**Last Updated**: 12-04-2026" line) so the metadata uses
unambiguous YYYY-MM-DD formatting.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/giving-readiness.md`:
- Around line 53-55: The document's "80G Application Process" section references
the obsolete "Form 10G"; update the text to reference the current forms "Form
10A/10AB" (replace any occurrences of "Form 10G" with "Form 10A/10AB" in the
"80G Application Process" checklist and related lines), adjust any accompanying
instructions or bullet points to match the 10A/10AB workflow and required
documents, and apply the same replacement/fix to the other affected ranges
mentioned (lines ~62-65) so the guidance points to the current official forms
and sources.
- Around line 157-160: Update the "Donor Privacy Policy" bullet to be
jurisdiction-aware: change the second sub-bullet to prioritize India (reference
DPDP Act/ Digital Personal Data Protection Act, 2023) and add a conditional note
that GDPR applies only "where EU data subjects are targeted"; keep remaining
bullets about storage/use and opt-in/opt-out intact so the section header "Donor
Privacy Policy" and its other bullets remain unchanged.
- Around line 348-352: The checklist's minimum launch gate currently allows
"/giving" to go live when "80G registration obtained (or application in process
with clear timeline)" — tighten this by requiring actual 80G approval for the
minimum gate: update the checklist item text to read "80G registration obtained"
(remove the "or application in process..." clause), update any related guidance
or conditional language that references the minimum launch requirement for the
/giving page, and ensure any downstream messaging or acceptance criteria that
checks the "80G" state uses the concrete "80G registration obtained" value
(search for the exact string "80G registration obtained (or application in
process with clear timeline)" and replace accordingly).

---

Nitpick comments:
In `@docs/giving-readiness.md`:
- Around line 365-369: Replace the ambiguous date string "12-04-2026" with the
ISO format "2026-04-12" in the document; update both occurrences shown in the
diff (the revision table entry containing "12-04-2026" and the "**Last
Updated**: 12-04-2026" line) so the metadata uses unambiguous YYYY-MM-DD
formatting.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 33d65027-f853-42ff-a8ba-5a271c0aa6d0

📥 Commits

Reviewing files that changed from the base of the PR and between c9b3504 and 09b4ae2.

📒 Files selected for processing (1)
  • docs/giving-readiness.md

Comment thread docs/giving-readiness.md
Comment thread docs/giving-readiness.md
Comment thread docs/giving-readiness.md
@vercel

vercel Bot commented Apr 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
alumni Ready Ready Preview, Comment Apr 12, 2026 9:20am

@MrImmortal09 MrImmortal09 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MrImmortal09 MrImmortal09 merged commit 96a276d into iiitl:main Apr 12, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Giving: readiness tracker (no code yet)

2 participants