Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve payout flow #1851

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Improve payout flow #1851

wants to merge 23 commits into from

Conversation

devkiran
Copy link
Collaborator

@devkiran devkiran commented Jan 2, 2025

This pull request includes multiple changes to the payout processing and email notification system. The most important changes include updating the payout confirmation process to support different payment methods, modifying email templates to dynamically format payout dates, and enhancing the UI for selecting payment methods.

Payout Processing Updates:

Email Template Modifications:

UI Enhancements:

Code Cleanup:

These changes collectively improve the flexibility and user experience of the payout processing system.

Copy link

vercel bot commented Jan 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
dub 🔄 Building (Inspect) Visit Preview Jan 9, 2025 1:16am

- Removed date formatting from the charge-succeeded webhook, directly using the payout period dates.
- Introduced a new email template for partner payout confirmation.
- Updated the partner payout sent email template to use Date objects for start and end dates, enhancing date handling.
- Modified the confirm payouts action to send confirmation emails to partners when payouts are processed via ACH, including relevant payout details.
- Cleaned up unused code in the payout invoice sheet component.
@devkiran devkiran marked this pull request as ready for review January 8, 2025 12:21
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