Skip to content

Fix broken 1DS package documentation links - #2797

Open
Jackson Weber (JacksonWeber) wants to merge 1 commit into
microsoft:mainfrom
JacksonWeber:jacksonweber-1ds-package-links
Open

Jackson Weber (JacksonWeber) wants to merge 1 commit into
microsoft:mainfrom
JacksonWeber:jacksonweber-1ds-package-links

Conversation

@JacksonWeber

Copy link
Copy Markdown
Contributor

Fix seven broken links in the 1DS Core and Post Channel READMEs.

  • Replace the removed enableDebugExceptions link with enableDebug.
  • Fix four cookie configuration anchors and correct the linked interface name.
  • Replace the nonexistent disableZip option with documentation for the supported zipPayload feature flag.
  • Replace the dead Common Schema Part A shortcut with the existing schema document.

No runtime changes.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings November 11, 2026 06:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues block approval of these documentation updates.

No open issues 🎉

What changed in this PR

Repairs documentation links in the 1DS Core and Post Channel READMEs and aligns configuration guidance with supported SDK options. No runtime behavior changes.

Changes:

  • Corrects the enableDebug link, cookie configuration anchors, and ICookieMgrConfig labels.
  • Replaces the nonexistent disableZip option with zipPayload feature-flag guidance.
  • Updates the Common Schema Part A link to the schema document.
File Description
shared/​1ds-core-js/​README.md Fixes debug and cookie configuration links and interface labels.
channels/​1ds-post-js/​README.md Corrects compression guidance and the Common Schema Part A link.

Review effort: Balanced

This branch has not been deployed

No deployments
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.

2 participants