Skip to content

feat(lazer/publisher-sdk): add asset state and governance messages #3140

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat(lazer/publisher-sdk): add asset state and governance messages #3140

wants to merge 3 commits into from

Conversation

tejasbadadare
Copy link
Contributor

@tejasbadadare tejasbadadare commented Oct 15, 2025 •
edited
Loading

Summary

  • Add Asset state and governance proto messages
  • Asset follows similar structure to Feed. The only static field is asset_id, the rest of the fields will be dynamic inside metadata.
  • There isn't a real need to scope Assets to specific shards since they are global reference data. As such, no shard related fields are included in Asset.
  • These values will be ignored by the governance handling code, so this is backward compatible and safe. We will only use these instructions when state and governance handling code has been updated to load these objects.

Rationale

  • Needed to add Assets to Lazer state

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@vercel Vercel
Copy link

vercel bot commented Oct 15, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
api-reference Ready Ready Preview Comment Oct 17, 2025 6:35pm
component-library Ready Ready Preview Comment Oct 17, 2025 6:35pm
developer-hub Ready Ready Preview Comment Oct 17, 2025 6:35pm
entropy-explorer Ready Ready Preview Comment Oct 17, 2025 6:35pm
insights Ready Ready Preview Comment Oct 17, 2025 6:35pm
proposals Ready Ready Preview Comment Oct 17, 2025 6:35pm
staking Ready Ready Preview Comment Oct 17, 2025 6:35pm

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

@tejasbadadare tejasbadadare changed the title feat: add asset state and governance messages feat(lazer/publisher-sdk): add asset state and governance messages Oct 16, 2025
@tejasbadadare tejasbadadare marked this pull request as ready for review October 16, 2025 18:47
darunrs
darunrs approved these changes Oct 16, 2025
View reviewed changes
Copy link
Contributor

@darunrs darunrs left a comment

Choose a reason for hiding this comment

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

Nice!

Riateche
Riateche approved these changes Oct 17, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Riateche Riateche Riateche approved these changes

@darunrs darunrs darunrs approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tejasbadadare @Riateche @darunrs