Update docfx.json #12
Conversation
✅ Validation status: passed
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
|
@Rick-Anderson What is this change for? |
|
@mairaw we need this change for the APEX meta-data initiative - right? |
|
Yes @Rick-Anderson but I'll leave some comments |
IIS-Administration/docfx.json
Outdated
| "breadcrumb_path": "/iis/breadcrumb/toc.json", | ||
| "_navPath": "/foo", | ||
| "_navRel": "/foo", | ||
| "searchScope": "" |
There was a problem hiding this comment.
why not add a search scope?
IIS-Administration/docfx.json
Outdated
| "fileMetadata": {}, | ||
| "globalMetadata": { | ||
| "breadcrumb_path": "/iis/breadcrumb/toc.json", | ||
| "_navPath": "/foo", |
There was a problem hiding this comment.
not familiar with these too _navPath and _navRel
| "_navRel": "/foo", | ||
| "searchScope": "" | ||
| }, | ||
| "fileMetadata": { |
There was a problem hiding this comment.
instead of adding filemetadata, add these in the globalmetadata bucket
| @@ -30,7 +30,20 @@ | |||
| "overwrite": [], | |||
| "externalReference": [], | |||
| "globalMetadata": {}, | |||
| "fileMetadata": {}, | |||
| "globalMetadata": { | |||
There was a problem hiding this comment.
you also need author and ms.author values and I'm seeing that all content is missing required metadata like title, description and ms.date.
IIS-Administration/docfx.json
Outdated
| @@ -30,7 +30,20 @@ | |||
| "overwrite": [], | |||
| "externalReference": [], | |||
| "globalMetadata": {}, | |||
| "fileMetadata": {}, | |||
| "globalMetadata": { | |||
| "breadcrumb_path": "/iis/breadcrumb/toc.json", | |||
There was a problem hiding this comment.
do they have this configured?
|
@shirhatti content published on docs.microsoft.com has some special metadata requirements for BI tracking purposes. We're running reports on all content published on docs now to find content that is missing or using wrong values for those and this repo was one that was flagged. You can find more info about metadata at https://review.docs.microsoft.com/en-us/help/contribute/metadata-attributes?branch=master |
IIS-Administration/docfx.json
Outdated
| "**/**.md": "iis-admin" | ||
| }, | ||
| "ms.topic": { | ||
| "**/**.md": "conceptual" |
There was a problem hiding this comment.
wondering if the content under the API path should be set as reference
|
@mairaw @Rick-Anderson Is this going to be merged? |
|
@Rick-Anderson , Maira is out. Can you, Scott or Tim get this?
|
✅ Validation status: passed
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
✅ Validation status: passed
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
|
This can be closed. It's replaced by #18. |
No description provided.