Skip to content

Conversation

@asurkov
Copy link
Collaborator

@asurkov asurkov commented Nov 28, 2025

No description provided.

@asurkov asurkov requested a review from sidvishnoi November 28, 2025 18:07
@asurkov asurkov self-assigned this Nov 28, 2025
Copy link
Member

@sidvishnoi sidvishnoi left a comment

Choose a reason for hiding this comment

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

In the extension's algorithm (specially when dealing with cross currency payments), there's a concept of "bucket" which fills according to user's rate of pay, and when there is enough in the bucket to be sent for next session in queue, the bucket empties and sends that amount to sessions. I don't see such thing in this algorithm here. Do you plan to handle cross currency scenarios later and keeping it simple for now?

@asurkov
Copy link
Collaborator Author

asurkov commented Dec 2, 2025

In the extension's algorithm (specially when dealing with cross currency payments), there's a concept of "bucket" which fills according to user's rate of pay, and when there is enough in the bucket to be sent for next session in queue, the bucket empties and sends that amount to sessions. I don't see such thing in this algorithm here. Do you plan to handle cross currency scenarios later and keeping it simple for now?

Yeah, the idea is to keep it intentionally basic for now and then implement all improvements on top of it.

@asurkov asurkov merged commit e0c2255 into main Dec 4, 2025
@asurkov asurkov deleted the paymentstreaming branch December 4, 2025 15:23
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.

3 participants