Skip to content

Conversation

nidhi-singh02
Copy link
Contributor

@nidhi-singh02 nidhi-singh02 commented Sep 9, 2025

Summary

Matching the content with the docs.

  • Added additional content for Integrate as searcher page.
  • HTTP API reference page has more details

Rationale

How has this been tested?

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

Copy link

vercel bot commented Sep 9, 2025

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

Project Deployment Preview Comments Updated (UTC)
component-library Ready Ready Preview Comment Sep 11, 2025 8:51am
developer-hub Ready Ready Preview Comment Sep 11, 2025 8:51am
5 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
api-reference Skipped Skipped Sep 11, 2025 8:51am
entropy-explorer Skipped Skipped Sep 11, 2025 8:51am
insights Skipped Skipped Sep 11, 2025 8:51am
proposals Skipped Skipped Sep 11, 2025 8:51am
staking Skipped Skipped Sep 11, 2025 8:51am

Copy link
Contributor

@alexcambose alexcambose left a comment

Choose a reason for hiding this comment

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

looks good overall, a few suggestions

@@ -1,204 +1,31 @@
---
title: "Error Codes"
title: "SVM Error Codes"
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe this should be Express Relay Error Codes instead? I feel like SVM Error Codes is too generic but feel free to ignore if it doesn't make sense

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As we are inside Express Relay, SVM Error Codes seems okay to me.

@vercel vercel bot temporarily deployed to Preview – staking September 11, 2025 08:14 Inactive
Comment on lines +27 to +31
<Callout variant="warning">
To simulate real performance conditions and activity, the staging/testing
environment for Solana is on the mainnet-beta network. This environment is not
for production use and should only be used for testing.
</Callout>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Using a Callout which seems much better.

@nidhi-singh02 nidhi-singh02 merged commit 4da35ce into main Sep 11, 2025
10 checks passed
@nidhi-singh02 nidhi-singh02 deleted the express-relay-enhancements branch September 11, 2025 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants