Skip to content

feat(lazer/sui): initialize #2964

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

feat(lazer/sui): initialize #2964

wants to merge 6 commits into from

Conversation

tejasbadadare
Copy link
Contributor

Summary

Initializes pyth_lazer contract modules and includes a rough draft of a function to parse & verify a Lazer payload based off this example.

How has this been tested?

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

Copy link

vercel bot commented Aug 19, 2025

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

Project Deployment Preview Comments Updated (UTC)
api-reference Ready Ready Preview Comment Aug 19, 2025 0:36am
component-library Ready Ready Preview Comment Aug 19, 2025 0:36am
developer-hub Ready Ready Preview Comment Aug 19, 2025 0:36am
entropy-explorer Ready Ready Preview Comment Aug 19, 2025 0:36am
insights Ready Ready Preview Comment Aug 19, 2025 0:36am
proposals Ready Ready Preview Comment Aug 19, 2025 0:36am
staking Ready Ready Preview Comment Aug 19, 2025 0:36am

Copy link
Collaborator

@ali-behjati ali-behjati left a comment

Choose a reason for hiding this comment

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

LGTM. Please address my comments before merging.

}

/// Create a new Feed with the specified parameters
public fun new(
Copy link
Collaborator

Choose a reason for hiding this comment

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

you need to make this public(crate) because you want when people get a Feed or Update object they be sure it's a valid Feed or Update.

/// Funding rate for derivative products (e.g., perpetual futures)
funding_rate: Option<Option<I64>>,
/// Timestamp when the funding rate was last updated
funding_timestamp: Option<Option<u64>>,
Copy link
Collaborator

Choose a reason for hiding this comment

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

we just recently added funding_rate_interval.

feeds: vector<Feed>,
}

public fun new(
Copy link
Collaborator

Choose a reason for hiding this comment

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

ditto

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