-
Notifications
You must be signed in to change notification settings - Fork 53
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
chore: clarify internal message flow #3000
Comments
This is a reasonable question, though one that will be more appropriately answered once Sync (and Store) is more integrated into our message propagation protocols. For now, it's simply an augmentation of Store and synced messages will only be accessible via store queries and/or store API |
@SionoiS - Isn't |
True, it's mostly a UX problem and expectation. |
ah okay , I wonder if it may help allowing |
From this point of view, IMHO, it is the missing SDK layer that could hide the "get me all messages I don't care from where" API. |
I realized that a user expecting a
Filter
subscribe to return messages fromSync
is going to be disappointed...More generally, do we need a higher level
get me all the new messages from wherever
API?If we think about the flow of messages, a Waku node can output msgs to Relay, Filter and Sync. Inputs would be Relay, Light push, Sync. It would make sense that a msg from Sync is not relayed but should it be used by Filter?
🤷
WDYT? @waku-org/nwaku-developers @jm-clius
The text was updated successfully, but these errors were encountered: