Skip to content

Conversation

@lsqproduction
Copy link
Collaborator

Summary

  • update Update Webhook references to use the internal docs path instead of the full alchemy.com URL
  • ensure Address Activity, NFT Activity, and Custom Webhooks tables all point at the same relative endpoint link

Testing

  • not run (not requested)

Codex Task

@github-actions
Copy link

github-actions bot commented Dec 15, 2025

🌿 Documentation Preview

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Dec 15, 2025, 11:06 PM

@lsqproduction lsqproduction changed the title Use relative update webhook endpoint link Use relative update webhook broken link Dec 15, 2025
@github-actions github-actions bot temporarily deployed to docs-preview December 15, 2025 23:05 Destroyed
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

| [Get all webhooks](/reference/team-webhooks) |
| [Delete webhook](/reference/delete-webhook) |
| [Update webhook status](/reference/update-webhook) |
| [Update webhook status](/data/webhooks/webhooks-api-endpoints/notify-api-endpoints/update-webhook) |

Choose a reason for hiding this comment

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

P1 Badge Restore update webhook link to existing reference page

The new link targets /data/webhooks/webhooks-api-endpoints/notify-api-endpoints/update-webhook, but there is no doc or redirect for this slug anywhere in the repo (find . -path '*notify-api-endpoints*' returns nothing except these three references), so clicking it will 404. The Notify OpenAPI still defines the update-webhook operation under the standard reference endpoints (src/openapi/notify/notify.yaml path /update-webhook), and the other webhook CRUD links in the same table still use /reference/.... This same broken link was also added to the Address Activity and NFT Activity tables.

Useful? React with 👍 / 👎.

@lsqproduction lsqproduction merged commit 5427875 into main Dec 15, 2025
8 of 9 checks passed
@lsqproduction lsqproduction deleted the codex/fix-broken-link-on-webhook-page branch December 15, 2025 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants