-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Edit Source link for Gitea, Forgejo, or other self-hosted online repository #12117
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
Comments
Hello @mlncn |
@EvanBinu i do not have the power to assign but i hope if you are able to add the code ReadTheDocs will merge it in! |
ok @mlncn |
Edit Source link for Gitea, Forgejo, or other self-hosted online repository readthedocs#12117
@mlncn |
@ericholscher Take a look at this one during PyConUSA sprint |
I looked at the PR here, and it seems more like theme documentation than RTD? Honestly we should probably just delete the guide that we have, since it isn't actually adding value compared to the theme docs here., |
What's the problem this feature will solve?
All code with repository hosting that supports online editing does not have to be on GitHub, GitLab / self-hosted GitLab, or Bitbucket. Gitea, Forgejo, and others exist and are increasingly popular.
Describe the solution you'd like
Extend the options documented on https://docs.readthedocs.com/platform/stable/guides/edit-source-links-sphinx.html to include "Other" with a
display_repo
setting that takes uses the following variables:repo_host
repo_user
(or org)repo_slug
(repository name as seen in the URL)repo_version
(branch)conf_src_path
(to take the place of hard-coded/src/
and/blob
path segments for our flexible custom repo)The existing variable would still be used:
conf_py_path
(appears right before pagename, defaults to/
)suffix
pagename
Alternative solutions
Override default templates, but it is time to support open projects as at least second-class citizens.
Additional context
Pointers for where to add this / how many default themes are supported that should get the above added to their templates would be greatly appreciated.
The text was updated successfully, but these errors were encountered: