-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Docs: Add documentation for CustomScript addon #12623
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: humitos <244656+humitos@users.noreply.github.com>
Documentation build overview
Show files changed (2 files in total): 📝 2 modified | ➕ 0 added | ➖ 0 deleted
|
Co-authored-by: humitos <244656+humitos@users.noreply.github.com>
humitos
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good with small changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the beginning of this file to add "Custom Script" to the initial list of features.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added "Custom Script" to the initial features list in commit 2abf4e4.
docs/user/addons.rst
Outdated
| // The event has already fired when custom scripts load. | ||
| // Check for window.ReadTheDocsEventData first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| // The event has already fired when custom scripts load. | |
| // Check for window.ReadTheDocsEventData first. | |
| // The event "readthedocs-addons-data-ready" has been already fired when this script is run. | |
| // We need to check for `window.ReadTheDocsEventData` first, and if it's available | |
| // use that data to call the handler. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated with the suggested comment text in commit 2abf4e4.
docs/user/addons.rst
Outdated
| handleReadTheDocsData(window.ReadTheDocsEventData.data()); | ||
| } | ||
| // Subscribe to the event for future data updates |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| // Subscribe to the event for future data updates | |
| // After that, we subscribe to the Read the Docs Addons event to access data | |
| // on future dispatchs (e.g. when a URL changes on a SPA) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated with the suggested comment text in commit 2abf4e4.
|
@copilot Fix feedback |
…de comments Co-authored-by: ericholscher <25510+ericholscher@users.noreply.github.com>
Fixed all feedback in commit 2abf4e4. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
docs/user/addons.rstwindow.ReadTheDocsEventDataflag for accessing addons dataOriginal prompt
CustomScript#11775💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.