Skip to content

264 Release PR#7326

Open
force2b wants to merge 7 commits into
mainfrom
feature/264
Open

264 Release PR#7326
force2b wants to merge 7 commits into
mainfrom
feature/264

Conversation

@force2b
Copy link
Copy Markdown
Contributor

@force2b force2b commented May 11, 2026

Critical Changes

Changes

  • NPSP Settings Accessibility Fixes

Issues Closed

Community Ideas Delivered

Features Intended for Future Release

New Metadata

Deleted Metadata

force2b and others added 7 commits May 10, 2026 14:30
…p bug (TDD)

Tests verify that monthly RDs with DayOfMonth=31/30/Last_Day starting in
short months (Feb/Apr/Jun) do not skip the current 31-day month when
calculating next donation date. Covers unit, integration, and batch paths.
Adds test for getVisualizedInstallments showing skipped installment when
viewed from a late day in the month (e.g., Mar 29 with Feb start DayOfMonth=31).
Also fixes April/June tests to use referenceDate=Jul 31 to actually trigger the bug.
Apply day-of-month correction before comparing against referenceDate so
the comparison uses the actual donation date (e.g., Jul 31) rather than
the clamped addMonths result (e.g., Jul 28). Extract adjustToTargetDayOfMonth
helper to avoid duplicating the correction logic.

W-19306508
…he 6-step algorithm

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…6508-monthly-rd-next-donation-date-fix

@W-19306508 Monthly RD Next Donation Date and Installment Fix
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.

2 participants