Skip to content

refactor: stream multi yield load#333

Merged
Philippoes merged 1 commit intomainfrom
feat/multi-yield-stream-load
Apr 2, 2025
Merged

refactor: stream multi yield load#333
Philippoes merged 1 commit intomainfrom
feat/multi-yield-stream-load

Conversation

@petar-omni
Copy link
Copy Markdown
Contributor

Summary

This PR refactors the multi-yield loading mechanism in the StakeKit Widget to implement a streaming approach. The changes optimize the loading and handling of yield opportunities by implementing reactive streams using RxJS.

Changes

  • Improved Performance: Optimized the loading of yield data by processing yields as they become available instead of waiting for all to load

Technical Details

  • Implemented RxJS streaming patterns for continuous data flow of yield opportunities
  • Added automatic refresh with configurable intervals for yield data
  • Improved handling of initial yield selection based on eligibility criteria

@petar-omni petar-omni requested a review from Philippoes April 1, 2025 14:44
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 1, 2025

🦋 Changeset detected

Latest commit: eb3cd05

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@stakekit/widget Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 1, 2025

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

Name Status Preview Comments Updated (UTC)
staging-widget ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 1, 2025 2:45pm
stakekit-widget ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 1, 2025 2:45pm

@Philippoes Philippoes merged commit cb905aa into main Apr 2, 2025
4 checks passed
@Philippoes Philippoes deleted the feat/multi-yield-stream-load branch April 2, 2025 12:03
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