Skip to content

feat: support monthly auto-invest plans - #560

Merged
overprodigy merged 16 commits into
Heliobond:mainfrom
Dan-tolu:fix/issue-404
Sep 2, 2026
Merged

feat: support monthly auto-invest plans#560
overprodigy merged 16 commits into
Heliobond:mainfrom
Dan-tolu:fix/issue-404

Conversation

@Dan-tolu

@Dan-tolu Dan-tolu commented Sep 1, 2026

Copy link
Copy Markdown

Summary

Adds a recurrence option to the bond deposit flow for dollar-cost averaging.

  • Lets investors opt into a monthly auto-invest plan for the same bond.
  • Limits the selected day to 1–28 so every month has a valid run date.
  • Shows the plan in the review step and persists it only after the initial deposit confirms.
  • Keeps the plan data local and explicit for the future scheduler/investment service.

Closes #404

Verification

  • git diff --check
  • Typecheck could not run because dependencies are not installed in the checkout (tsc: command not found).

olutundee and others added 8 commits August 31, 2026 16:55
- Listen for document visibilitychange events in HelioWebGL
- Stop animation and switch Canvas frameloop to demand when tab is hidden
- Resume continuous rendering when tab returns to visible
- Add unit tests for HelioWebGL tab visibility and motion behavior
- Clean up upstream syntax and type errors across codebase
@Dan-tolu
Dan-tolu requested a review from dadadave80 as a code owner September 1, 2026 18:40
@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

@zeemscript is attempting to deploy a commit to the David Dada's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Sep 1, 2026

Copy link
Copy Markdown

@Dan-tolu Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@sshdopey sshdopey 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.

This is a solid start on the monthly auto-invest feature! I love the clean RecurringInvestmentOptions component and the localStorage persistence for plans. The day-of-month limit to 1–28 is a thoughtful touch. Keep it up — once the scheduler/service is wired in, this will be a great DCA tool. Thanks for the contribution! ❤️

@sshdopey

sshdopey commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Approved! 🎉 I couldn't auto-merge this just yet (it may need a rebase or have a check still running). A maintainer can merge it whenever it's ready.

@Dan-tolu

Dan-tolu commented Sep 2, 2026

Copy link
Copy Markdown
Author

@sshdopey @Dan-tolu Conflict resolved and pushed in commit a2a8b8ca14a16b05f2a80585a204960d1f716c1c. The rebased branch preserves the contributor’s remote commit and removes the invalid setTxHash reference while retaining recurring-plan persistence. git diff --check passes and the remote branch is confirmed at this SHA. Please rerun checks and merge when ready.

@sshdopey sshdopey 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.

This is a solid effort toward supporting monthly auto-invest plans! The new RecurringInvestmentOptions component, localStorage persistence, and integration into the deposit flow directly address the issue. I appreciate the careful day-of-month limiting and the clear plan for future scheduler work. The diff also includes many helpful fixes and refactors. Great job! Keep iterating and we'll get this polished together. 💚

@sshdopey

sshdopey commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approved! 🎉 I couldn't auto-merge this just yet (it may need a rebase or have a check still running). A maintainer can merge it whenever it's ready.

@overprodigy
overprodigy merged commit e29e0f4 into Heliobond:main Sep 2, 2026
2 of 3 checks passed
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.

enhancement: Support recurrence/auto-invest to buy same bond monthly

8 participants