Skip to content

DO NOT MERGE: Update facilitator-fees FAQ default for per-person membership (#2095) - #2137

Draft
maebeale wants to merge 3 commits into
mainfrom
maebeale/issue-2095-fix
Draft

DO NOT MERGE: Update facilitator-fees FAQ default for per-person membership (#2095)#2137
maebeale wants to merge 3 commits into
mainfrom
maebeale/issue-2095-fix

Conversation

@maebeale

@maebeale maebeale commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

🤖 suggested review level: 3 Read 📖 FAQ default-copy change now interpolates a live constant

⚠️ DO NOT MERGE — should land only when the per-person membership feature launches.

Closes #2095

Why

What

  • Reworked the answer in BuiltinCallouts::FAQS to describe a per-facilitator annual membership fee that renews automatically, using membership terminology to match the post-Rename dues subscription to membership #2106 rename.
  • Amount is interpolated from Membership::ANNUAL_COST_CENTS (currently $25) rather than hard-coded, so the copy can't silently drift from the real figure years from now.
  • This is the default seeded onto new FAQ callout cards (admin-editable per event) and the FAQ page fallback — forward-looking only, no backfill of existing/edited cards.

Note

  • The value is captured at class-load time, so a change to ANNUAL_MEMBERSHIP_CENTS takes effect on the next deploy (not live per-request) — fine for static default copy.

Outstanding (why DNM)

  • Merge to coincide with the membership launch so the FAQ doesn't advertise a model that isn't live (still gated behind Membership.enabled?).

🤖 Generated with Claude Code

The old copy described a $100 per-program membership that covered all
facilitators in a program. PR #2094 replaces that with a per-person,
auto-renewing annual dues subscription, so the default FAQ answer is now
inaccurate. Rework the answer to describe per-facilitator annual dues.

Amount is intentionally omitted pending AWBW confirmation of the final
per-person figure (production dues are still disabled).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 9, 2026 20:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings August 9, 2026 21:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Derive the amount from the constant so the default copy can't drift from
the real per-person figure years from now, and use membership terminology
to match the post-#2106 rename.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings August 9, 2026 21:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@maebeale maebeale changed the title DO NOT MERGE: Update facilitator-fees FAQ default for per-person dues (#2095) DO NOT MERGE: Update facilitator-fees FAQ default for per-person membership (#2095) Aug 9, 2026
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.

Update callout faq defualt text when the new person subscription launches

2 participants