Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

chore(api-builder): support API doc language agnostic URIs #1849

Closed

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jul 8, 2016

This PR is in parts (and best explained and reviewed that way, in part because of the rename):

chore(api-builder): rename linkDevGuide to linkDocs

Rename the specialized API docs @linkDevGuide tag to @linkDocs since it is meant to be used to refer to any angular.io chapter, not just the developer guide.

chore(api-builder): let linkDocs links resolve to proper language

The old linkDevGuide required URIs to be given in the form 'ts/latest/guide/architecture' which forced all language versions of the API pages to refer to the ts versions of the doc pages they linked to.

This PR fixes that problem. URIs are now given in the form 'guide/architecture' and then rendered properly for each language. Of course, when needed, it is still possible to give a language specific URI.

chalin added 3 commits July 12, 2016 13:40
Rename the specialized API docs `@linkDevGuide` tag to `@linkDocs`
since it is meant to be used to refer to all angular.io docs, not just
the developer guide.
The old linkDevGuide required URIs to be given in the form
'ts/latest/guide/architecture' which forced all language versions of
the API pages to refer to the ts versions of the doc pages they linked
to.

This PR fixes that problem. URIs are now given in the form
'guide/architecture' and then rendered properly for each language. Of
course, when needed, it is still possible to give a language specific
URI.
@chalin chalin force-pushed the chalin-chore-api-doc-builder-0707 branch from a999c17 to cd099b5 Compare July 12, 2016 20:41
@wardbell
Copy link
Contributor

I will squash and merge (no one cares about the steps)

@wardbell wardbell closed this in 4da23bf Jul 14, 2016
@chalin chalin deleted the chalin-chore-api-doc-builder-0707 branch July 14, 2016 22:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants