Skip to content

feat: use cron for monthly ZapPlanner payments#1722

Merged
rolznz merged 2 commits intomasterfrom
chore/zapplanner-monthly-use-cron
Sep 16, 2025
Merged

feat: use cron for monthly ZapPlanner payments#1722
rolznz merged 2 commits intomasterfrom
chore/zapplanner-monthly-use-cron

Conversation

@rolznz
Copy link
Copy Markdown
Contributor

@rolznz rolznz commented Sep 16, 2025

Closes #1643

This way we ensure payments are made exactly monthly, both for the normal ZapPlanner app and for the support payments.

Copy link
Copy Markdown
Contributor

@reneaaron reneaaron left a comment

Choose a reason for hiding this comment

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

Looks good to me, although that means the payment isn't executed right after creation, right? (iirc there is some toast "Your payment is scheduled to be executed immediately")

How does that play together with the monthly budgets in Alby Hub? Is a monthly budget reset every 1st of the month? 🤔

@rolznz
Copy link
Copy Markdown
Contributor Author

rolznz commented Sep 16, 2025

@reneaaron thanks for the review

although that means the payment isn't executed right after creation, right? (iirc there is some toast "Your payment is scheduled to be executed immediately")

Yeah, I changed it to show a different message on the toast for this case.

How does that play together with the monthly budgets in Alby Hub? Is a monthly budget reset every 1st of the month? 🤔

This exactly fixes the problem, because hub budgets are reset on the 1st of the month. Now with cron we can ensure we only make exactly one payment per month, rather than guessing by setting an interval of 31 days.

Copy link
Copy Markdown
Member

@im-adithya im-adithya left a comment

Choose a reason for hiding this comment

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

utACK

@rolznz rolznz merged commit 6fb08d2 into master Sep 16, 2025
11 checks passed
@rolznz rolznz deleted the chore/zapplanner-monthly-use-cron branch September 16, 2025 13:50
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.

Add cron support for ZapPlanner payments

3 participants