Skip to content

Dashboard: joined-to-date card and growth chart now count the same students - #303

Merged
peiraisotta merged 1 commit into
WordPress:trunkfrom
peiraisotta:fix/joined-to-date-matches-growth-chart
Nov 9, 2026
Merged

peiraisotta merged 1 commit into
WordPress:trunkfrom
peiraisotta:fix/joined-to-date-matches-growth-chart

Conversation

@peiraisotta

Copy link
Copy Markdown
Contributor

The growth chart used its own status allowlist and missed the newer tracks (Intro to Contribution, Developer, Designer, Foundations), reading 893 against the card's 1034. Both now share one has_joined() predicate, Paused and Fail count as joined in both, and the build warns if the totals ever differ beyond pre-launch or future-dated records. Tests added.

…chart

The growth chart kept its own allowlist of "joined" statuses and missed the
newer tracks (Intro to Contribution, Developer, Designer, Foundations), so it
read 893 against the card's 1034. Both now use one has_joined() predicate:
every status except Interested, Not moving forward and SPAM. Paused and Fail
count as joined in both. The build warns if the two totals ever differ by more
than the pre-launch or future-dated records the chart leaves out.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SL7xkuA9nu4T8BeMMRX6Av
@peiraisotta
peiraisotta merged commit 6d74b58 into WordPress:trunk Nov 9, 2026
1 check passed
@peiraisotta
peiraisotta deleted the fix/joined-to-date-matches-growth-chart branch November 9, 2026 16:18
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