docs: fix stale links and grammar, remove obsolete doc files - #3734
Conversation
- Point the two angular.io links (compat.md, compat/auth/router-guards.md) at angular.dev, the current home of the Angular documentation. - Fix the App Check Getting Started link to the reCAPTCHA v3 page so it matches the ReCaptchaV3Provider used in the sample. - zones.md: the Warn logging-level bullet described high-risk APIs as "are called" where it means "are logged". - Correct "Provide a" to "Provide an" in the Auth, Analytics, and App Check guides. - Remove docs/firebase.json (a dead Firebase Hosting config for a documentation site that no longer exists) and docs/install-angular-cli-windows10.md (an obsolete, orphaned guide that recommends the npm-deprecated windows-build-tools).
tyler-reitz
left a comment
There was a problem hiding this comment.
All correct. The App Check link change in particular: the sample uses ReCaptchaV3Provider, which corresponds to the recaptcha-provider page rather than recaptcha-enterprise-provider, so the link now matches the code beside it.
Both deletions are justified. docs/firebase.json is a dead hosting config, and the Windows CLI guide points at windows-build-tools, which is archived — leaving it in place is worse than having no page at all.
Merge-order note: this and #3732 both touch docs/app-check.md and docs/auth.md with adjacent hunks. No overlap in substance, but whichever lands second should be rebased and re-skimmed rather than merged on a stale green.
|
Thanks Tyler. The |
Description
Sweeps a set of small documentation defects: stale links, one incorrect reference link, a wording fix, grammar, and two obsolete files.
angular.iolinks (docs/compat.md,docs/compat/auth/router-guards.md) atangular.dev.ReCaptchaV3Providersample.docs/zones.md: correct the "Warn" logging-level bullet from "are called" to "are logged".docs/firebase.json(a dead Firebase Hosting config) anddocs/install-angular-cli-windows10.md(an obsolete, orphaned CLI install guide that recommends a deprecated tool).Documentation only, no source or API changes.
Checklist
yarn install,yarn testrun successfully?: not applicable, no code changedFixes #3733