Preview - #59
Open
lyzhang0113 wants to merge 21 commits into
Open
Conversation
Repo-wide quality pass covering license/, mrz-scanner/, and barcode-reader/ (general, mobile, server, web): repoint links broken by the mobile/web directory reorgs, fix H1s that didn't match their index.md link text or frontmatter, remove redundant/duplicate sections, and reword leftover Q&A-snippet phrasing (unanswered "this"/"Yes—" references) into plain statements. No technical facts, code samples, or archived version-pinned content were altered. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Closes #32 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The intro text already said the filter is enabled by default; the code comment and settings table still said false. Starting from v4.0.0 the default is true, so align both to match. Also fixes a stray extra backtick breaking the code fence's closing line. Closes #33 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…sion FAQs The npm package backing DBR-JS has been renamed across major versions: dynamsoft-javascript-barcode (v9) -> dynamsoft-barcode-reader (v10) -> dynamsoft-barcode-reader-bundle (v11+). The v10 section previously used `npm show <pkg> version`, which queries the npm registry's latest published version rather than what's actually installed; switched both files to `npm list <pkg>`, which reports the installed version. Also fixed the v9 command's en-dash typo (`npm –v` -> was never valid) and split the web/debug FAQ into separate v9/v10/v11+ sections since v10 and v11 now use different package names. Closes #34 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Closes #35 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- barcode-reader/server/archive/unable-to-scan-aztec-code.md duplicated the live web-edition page (barcode-reader/web/scan-setting/unable-to-scan-aztec-code.md) and didn't belong in the server FAQ. Removed the file and its entry in server/configuration/index.md. - barcode-reader/web/archive/use-of-onFrameRead-and-onUnduplicateRead.md stays in the repo as historical/archived content, but is no longer linked from web/configuration/index.md. Its filename typo (onUnduplicateRead vs onUniqueRead) isn't worth fixing now that it's unlinked archived content. Closes #41, closes #42 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
barcode-reader/{mobile,server,web}/archive/ hold historical,
version-pinned content (e.g. v9.6.40, v10.4.2000 snapshots) that's
no longer linked from any live index or sidebar. Excluding them from
the build means they're kept in the repo for reference but are no
longer built, hosted, or listed in sitemap.xml.
Verified locally with `jekyll build`: the three directories are
absent from the generated _site output and from sitemap.xml, and the
two edited index pages (server/configuration, web/configuration)
render with no dangling references.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Exclude archived FAQ content from Jekyll build; clean up archive references
README.md was a full Jekyll page (frontmatter, inline CSS grid) duplicating the homepage (index.md), unlinked from anywhere in the repo, and shipped in the site build/sitemap despite that. Replaced it with a plain Markdown README describing the repo for contributors (structure, build/deploy process, link-checking), and added it to _config.yml's exclude list so Jekyll no longer builds or lists it in sitemap.xml. Verified locally with `jekyll build`: README.md is absent from the generated _site output and from sitemap.xml. Closes #53 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Dynamic Web TWAIN isn't part of Capture Vision, and its FAQ content isn't in this repo — only linked to from the homepage. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
README.md should describe the repo for humans; the article structure, frontmatter, linking, and archive-directory conventions are actionable rules for whoever (human or agent) is writing/wiring FAQ content, so they now live in AGENTS.md instead. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
H1/title used to say "Secure Your License Key by Managing Domain Binding," while both index.md entries linking to it said "change domain binding of product key" — picked the latter phrasing (it matches the filename and the actual step-by-step content) and applied it consistently to the H1, frontmatter title/description, and both index link texts. Filename already matched this phrasing, so no rename was needed. Closes #47 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Closes #46 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The page's title/H1/description/body are all specifically about checking camera *permissions* programmatically, not general camera availability. Updated both index.md link texts that pointed to it (web/camera/index.md and web/index.md) to match. Closes #43 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
# Conflicts: # barcode-reader/web/configuration/pop-up-window-during-trail-process.md
- mrz-scanner/general/ui-customization.md: close the example fence with three backticks instead of four (broke rendering for the rest of the page). - Align frontmatter title/description with the H1 in five files where this PR updated the H1 but left the old phrasing in metadata: license/how-hardware-is-bind-to-license.md, license/dbr-free-trial.md, barcode-reader/server/configuration/non-english-characters.md, barcode-reader/web/camera/stop-camera-to-open-right-away.md, barcode-reader/general/avoid-incorrect-results.md. - barcode-reader/mobile/configuration/no-camera-enhancer.md: description said "DBR Android FAQs" even though the page covers Android and iOS. - barcode-reader/mobile/index.md: link text still said generic "Template Support"; match the corrected label already used in scan-setting/index.md. Found by Copilot's automated PR review and a follow-up manual pass. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
FAQ docs quality pass: fix broken links, mismatched H1s, orphaned phrasing
license/*.md is shared content that gets duplicated at build time under /faq/barcode-reader/license/ and /faq/mrz-scanner/license/, each with product-scoped SEO metadata (_data/license_seo_overrides.yml) and correctly highlighted in their product's sidebar. The bare /faq/license/* path Jekyll also generates from the file's literal repo location isn't linked from anywhere on the site, and since it belongs to neither product, the sidebar can't highlight anything for it and falls back to rendering the entire unscoped DBR+MRZ tree. Nothing links to the bare path internally, so exclude it from sitemap.xml (same pattern as the existing Hide_Tree_Page.html scope) to stop it from being crawled. The page itself is unaffected and still builds normally for anyone who navigates there directly. Verified with a local Jekyll build: /faq/license/* entries are gone from sitemap.xml, while the two product-scoped duplicates remain.
Exclude bare /faq/license/ path from sitemap
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
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.
This pull request introduces substantial improvements to the documentation and FAQ content organization for the Dynamsoft FAQ site. The main focus is on clarifying article authoring conventions, updating navigation and linking practices, and refining several FAQ articles for accuracy and consistency. Additionally, the Jekyll site configuration is updated to exclude archived content and the plain
README.mdfrom the build. Below are the most important changes:Documentation and Authoring Guidelines
AGENTS.mddetailing FAQ article structure, linking, directory conventions, and archiving rules for both AI agents and human contributors.README.mdto focus on repo purpose, article authoring, build/deploy process, and link checking; removed old frontmatter and homepage grid.Site Configuration
_config.ymlto exclude allarchivedirectories andREADME.mdfrom the Jekyll build, ensuring only current content is hosted and indexed.FAQ Content Corrections and Improvements
barcode-reader/general/avoid-incorrect-barcode-results.md.barcode-reader/general/check-current-version.md. [1] [2]These changes collectively improve the maintainability, clarity, and navigability of the FAQ site.
Issues fixed
preview)Also fixed, found by Copilot's automated review of #30 (no standalone issues filed):
Not included in this PR (still open, need a human decision or fact-check): #31, #44, #45, #48, #51, #52, #54.