fix(docs): remove duplicate internal tables section & sidebar header link#4157
Open
AnnasMazhar wants to merge 1 commit into
Open
fix(docs): remove duplicate internal tables section & sidebar header link#4157AnnasMazhar wants to merge 1 commit into
AnnasMazhar wants to merge 1 commit into
Conversation
Two fixes for docs duplication introduced by PR dlt-hub#4028: 1. Remove duplicate 'dlt's internal tables' section from destination-tables.md. This section was a verbatim copy of 'Internal dlt tables' in dataset-access/dataset.md (the canonical source). Removes 64 lines of duplicate content. 2. Fix 'Configuration & secrets' sidebar category header linking to its own page (a top-level header with a doc link). Removed the property and added the same doc as an 'Overview' item inside the category, so clicking the header expands the category instead of navigating to a page. Closes dlt-hub#4125
AnnasMazhar
force-pushed
the
docs/revert-duplicate-docs-4125
branch
from
7cabaaf to
c5d9258
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #4125
Changes:
destination-tables.md — Removed the duplicate "dlt's internal tables" section (64 lines). This was a verbatim copy of the "Internal
dlttables" section indataset-access/dataset.md(the canonical source), introduced by the re-indexing in (fix) restored orphaned content after docs restructuring #4028.sidebars.js — Converted the "Configuration & secrets" category header from a linked doc to a collapsible header with an explicit "Overview" item. Previously, clicking the category header navigated to
general-usage/credentials/index— a top-level header should not have an associated page.Verification:
_dlt_loads,_dlt_pipeline_state,_dlt_version, same schemas, same descriptions)dlthub.md(new file in (fix) restored orphaned content after docs restructuring #4028) checked for duplicates — none found, distinct content fromtransformations/index.md