Skip to content

Conversation

@Joe1the2creator33
Copy link

@Joe1the2creator33 Joe1the2creator33 commented Jun 8, 2025

Fork for proposed difference to insured protection

Summary

Screenshots (optional)

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

Fork for proposed difference to insured protection
@Joe1the2creator33 Joe1the2creator33 requested review from a team and dcpena as code owners June 8, 2025 08:03
@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented Jun 8, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.


PR Change Summary

Updated the documentation for the Change Super Administrator feature to include a new database connection example using Hyperdrive.

  • Introduced a new example for connecting to a database using Hyperdrive.
  • Added a sample SQL query to demonstrate data retrieval.
  • Included error handling for database connection issues.

Modified Files

  • src/content/docs/fundamentals/account/change-super-admin.mdx

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

Copy link
Contributor

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

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

💡 To request another review, post a new comment with "/windsurf-review".

Comment on lines +1 to +22
import postgres from 'postgres';

export default {
async fetch(request, env, ctx): Promise<Response> {
// Hyperdrive provides a unique generated connection string to connect to
// your database via Hyperdrive that can be used with your existing tools
const sql = postgres(env.HYPERDRIVE.connectionString);

try {
// Sample SQL query
const results = await sql`SELECT * FROM pg_tables`;

// Close the client after the response is returned
ctx.waitUntil(sql.end());

return Response.json(results);
} catch (e) {
return Response.json({ error: e instanceof Error ? e.message : e }, { status: 500 });
}
},

} satisfies ExportedHandler<{ HYPERDRIVE: Hyperdrive }>;---
Copy link
Contributor

Choose a reason for hiding this comment

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

This PR appears to be adding code for a Hyperdrive database connection to a documentation file about changing Super Administrators. This code doesn't seem to belong in this documentation file and would break the MDX formatting. The file should only contain documentation content related to changing Super Administrators in Cloudflare accounts.

@Joe1the2creator33
Copy link
Author

Fork for proposed difference to insured protection

Summary

Screenshots (optional)

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@Joe1the2creator33
Copy link
Author

500

@cloudflare cloudflare locked as spam and limited conversation to collaborators Jun 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants