Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Oct 17, 2025

Description

Added a "Why Update Prices" section to the EVM Pull Integration guide that explains why price updates are required in Pyth's pull-based oracle model. This addresses a common point of confusion for developers new to Pyth who may be accustomed to traditional push oracles.

The section includes:

  • An explanation of Pyth's pull-based oracle model
  • Three key advantages (lower costs, lower latency, flexibility)
  • The three required steps in the Pull integration pattern
  • A warning callout highlighting the StalePrice error that occurs when prices aren't updated

Type of Change

  • New Page
  • Page update/improvement
  • Fix typo/grammar
  • Restructure/reorganize content
  • Update links/references
  • Other (please describe):

Areas Affected

  • pages/price-feeds/use-real-time-data/pull-integration/evm.mdx - Added new "Why Update Prices" section with explanatory content and warning callout

Checklist

  • I ran npm run lint to check for linting errors (passed with no errors)
  • I have reviewed my changes for clarity and accuracy
  • All links are valid and working
  • Images (if any) are properly formatted and include alt text (N/A)
  • Code examples (if any) are complete and functional (N/A - no new code examples)
  • Content follows the established style guide
  • Changes are properly formatted in Markdown
  • Preview renders correctly in development environment

Related Issues

N/A

Additional Notes

Structure:

  • Section heading: "## Why Update Prices"
  • Regular paragraph text explaining the pull-based model and its advantages
  • Warning callout containing only the important note about StalePrice errors

This structure ensures the information flows naturally while emphasizing the critical warning about price staleness.

Contributor Information

- Added a warning callout before the 'Write Contract Code' section
- Explains the pull-based oracle model concept
- Highlights advantages: lower costs, lower latency, flexibility
- Details the three steps required in Pull integration
- Warns about StalePrice error when prices aren't updated

Co-Authored-By: Nidhi Singh <[email protected]>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@vercel
Copy link

vercel bot commented Oct 17, 2025

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

Project Deployment Preview Comments Updated (UTC)
docs Ready Ready Preview Comment Oct 17, 2025 0:35am
documentation Error Error Oct 17, 2025 0:35am


## Why Update Prices?

<Callout type="warning" emoji="⚠️">
Copy link
Member

@aditya520 aditya520 Oct 17, 2025

Choose a reason for hiding this comment

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

Either you can place the title inside the callout or you remove the callout and put the text as a paragraph.

Copy link
Contributor Author

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Thanks for the feedback! I've moved the title inside the callout to address the redundancy. The callout now starts with "Why Update Prices?" as the first line.

@nidhi-singh02
Copy link
Contributor

add a section with title Why Update Prices and add the Important note in the call out. Have rest of the content in the section.

Per feedback from @nidhi-singh02:
- Added 'Why Update Prices' as a section heading
- Moved explanatory content outside the callout
- Kept only the Important note about StalePrice errors in the callout

Co-Authored-By: Nidhi Singh <[email protected]>
@devin-ai-integration devin-ai-integration bot changed the title Add callout explaining why price updates are needed in Pull integration Add 'Why Update Prices' section to EVM Pull Integration guide Oct 17, 2025
@nidhi-singh02 nidhi-singh02 enabled auto-merge (squash) October 17, 2025 12:40
@nidhi-singh02 nidhi-singh02 merged commit ac2f3d1 into main Oct 17, 2025
4 of 5 checks passed
@nidhi-singh02 nidhi-singh02 deleted the devin/1760682223-add-price-update-callout branch October 17, 2025 12:46
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.

3 participants