Skip to content
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

Publish Pando metadata payload as Any instead of Bytes #417

Merged
merged 1 commit into from
May 4, 2022

Conversation

masih
Copy link
Member

@masih masih commented May 4, 2022

Update to the latest Pando which changes the metadata payload type from
Bytes to Any. Reflect the changes in dealbot publisher, such that:

  • the payload is an IPLD node that is made up of a single link that
    points to the actual data
  • the GraphSync server exposed by the dealbot publisher allows the link
    to be traversed as before, so that Pando can fully sync the data from
    dealbot as long as all links in payload are explored recursively.

Relates to #416

@masih masih force-pushed the masih/pando-payload-any branch from 251e8bc to 33bfdbe Compare May 4, 2022 09:36
@masih masih requested a review from willscott May 4, 2022 09:36
Update to the latest Pando which changes the metadata payload type from
`Bytes` to `Any`. Reflect the changes in dealbot publisher, such that:
* the payload is an IPLD node that is made up of a single link that
  points to the actual data
* the GraphSync server exposed by the dealbot publisher allows the link
  to be traversed as before, so that Pando can fully sync the data from
  dealbot as long as all links in payload are explored recursively.

Relates to #416
@masih masih force-pushed the masih/pando-payload-any branch from 33bfdbe to 40dca51 Compare May 4, 2022 09:50
@masih masih merged commit 3f74c43 into main May 4, 2022
@masih masih deleted the masih/pando-payload-any branch May 4, 2022 10:21
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