-
Notifications
You must be signed in to change notification settings - Fork 860
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Improve this Document" link not appearing for Gitlabs. #1144
Comments
I found GitLab has the same URL pattern with GitHub, you can try adding |
@superyyrrzz thanks for the prompt reply. I tried that as well but it does not work.
|
I think your docfx.json is right, and local branch is also OK. Here're some other steps that may help narrow down the problem.
|
@superyyrrzz I am using docfx.msbuild version 2.4.0 I have now removed _gitContribute and newFileRepository but result is still the same. |
@superyyrrzz I found out that I was using the docfx.msbuild which was wrong package. I have installed docfx.console nuget 2.10.2 and now "Improve this doc" and "View Source" are appearing. When I click on "View Source", it leads me to 404 because it builds the link with the local branch name. |
For a reference page, if it hasn't an overwrite document, "Improve this doc" will link to a URL that help create a file to overwrite this page. If the remote branch exists, you can see what this URL does, instead of a 404. |
For me these two settings worked:
|
In our case, it worked with false as a boolean, not a string. "globalMetadata": {
"_appTitle": "JJMasterData",
"_disableContribution": false,
"_gitContribute": {
"branch": "master"
},
"_gitUrlPattern": "github"
}, |
I am working on a documentation project for GitLabs hosted project.
URL format is https://git.companyname.com
How to make the link work?
part of docfx.json is as shown below.
"globalMetadata": {
"_enableSearch": "true",
"_gitContribute": {
"repo": "https://git.companyname.com/dev/Development.git",
"branch": "master",
"path": ""
},
"newFileRepository": {
"branch": "master"
}
},
The text was updated successfully, but these errors were encountered: