Skip to content

Conversation

@markulrich
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Sep 25, 2025

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

Project Deployment Preview Updated (UTC)
donate-button Ready Ready Preview Sep 25, 2025 4:10am

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR simplifies donation frequency handling by defaulting to one-time donations when no frequency is selected. This change removes the need for complex frequency validation logic in the UI components.

  • Default frequency to DonationFrequency.OneTime when no frequency is provided
  • Simplify submit button disabled state logic by removing frequency-related validation
  • Remove unused imports and variables related to frequency checking

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
packages/donate-button-v4/src/helpers/constructDonateUrl.ts Adds default one-time frequency assignment using logical OR assignment operator
packages/donate-button-v4/src/components/widget/components/PaymentProcess/DefaultFlow/index.tsx Removes frequency validation logic and unused imports, simplifying submit button state

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@markulrich markulrich enabled auto-merge (squash) September 25, 2025 04:10
@markulrich markulrich disabled auto-merge September 25, 2025 04:20
@markulrich markulrich merged commit 332e653 into main Sep 25, 2025
6 checks passed
@markulrich markulrich deleted the mu/allow-no-freq branch September 25, 2025 04: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