docs: fix stale links and grammar, remove obsolete doc files - #3734
Open
armando-navarro wants to merge 1 commit into
Open
docs: fix stale links and grammar, remove obsolete doc files#3734armando-navarro wants to merge 1 commit into
armando-navarro wants to merge 1 commit into
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).
armando-navarro
added
comp: docs
tyler-reitz
approved these changes
Aug 4, 2026
tyler-reitz
left a comment
tyler-reitz
left a comment
Collaborator
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.
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.
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