Add receipt blob to get message since #229
Conversation
🦋 Changeset detectedLatest commit: 9e61264 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
👋 carte7000, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
|
The latest Buf updates on your PR. Results from workflow Regenerate Protobuf Files / buf (pull_request).
|
|
|
||
| message GetMessagesSinceResponse { | ||
| repeated VerifierResult results = 1; | ||
| repeated MessageWithVerifierResult results = 1; |
There was a problem hiding this comment.
I think we're moving to the pattern where there's a GetMessagesSince for message discovery which only provides the message, then a k-v lookup which provides the verifier results right?
Should this just be Message and receipt blobs without ccv_data?
There was a problem hiding this comment.
I think we can do that but I would rather do it when indexer is ready. Removing it now would be a big breaking change
3 participants
No description provided.