Codeberg/Documentation
42
199
Fork
You've already forked Documentation
161

Add Documentation for integrating with Read the Docs #289

Merged
fnetX merged 5 commits from :readthedocs into main 2023年02月24日 20:37:25 +01:00
Contributor
Copy link

This adds a Documentation for connecting Codeberg to Read the Docs. English is not my native Language, so there may be a few things that could be written better.

This adds a Documentation for connecting Codeberg to [Read the Docs](https://readthedocs.org/). English is not my native Language, so there may be a few things that could be written better.
@ -0,0 +5,4 @@
parent: Integrations
---
This article will guide you through integrating Codeberg with [Read the Docs](https://readthedocs.org/) to allow automatic build of your Documentation. It will show you, how to connect Read the Docs to a Codeberg Repository. To learn how Read the DOcs works, take a loot at the [official Documentation](https://docs.readthedocs.io).
Owner
Copy link

*Docs

*Docs

loot -> look

loot -> look
JakobDev marked this conversation as resolved
@ -0,0 +14,4 @@
1. On Read the Docs click "Import a Project"
2. If you have connected Read the Docs with some Provider e.g. GitHub, you see a list of your Repos at that provider. Ignore it and click "Import Manually"
3. You should see this screen:
Owner
Copy link

Step three is not really a step, is it? It interrupts the flow IMHO. Please also ensure that you write in a way that allows to follow even without pictures (they might be disabled in low-spec machines or not accessible to readers with limited or missing eye vision).

You can use screenshots to better convey your information, though. But I think they should be described differently.

Step three is not really a step, is it? It interrupts the flow IMHO. Please also ensure that you write in a way that allows to follow even without pictures (they might be disabled in low-spec machines or not accessible to readers with limited or missing eye vision). You can use screenshots to better convey your information, though. But I think they should be described differently.
Author
Contributor
Copy link

I don't know how to describe this better, but the Screenshot is optional.It just for people to confirm they are on the right screen. If you just follow the steps, everything should be fine.

I don't know how to describe this better, but the Screenshot is optional.It just for people to confirm they are on the right screen. If you just follow the steps, everything should be fine.
Contributor
Copy link

I added some proposals for the alt-attribute of the image tag.
You may use or adapt them as you see fit.

I added some proposals for the alt-attribute of the image tag. You may use or adapt them as you see fit.
Owner
Copy link

Thank you.

Thank you.
@ -0,0 +7,4 @@
This article will guide you through integrating Codeberg with [Read the Docs](https://readthedocs.org/) to allow automatic build of your Documentation. It will show you, how to connect Read the Docs to a Codeberg Repository. To learn how Read the DOcs works, take a loot at the [official Documentation](https://docs.readthedocs.io).
> Gitea/Forgeo is currenly not official supported by Read the Docs, so this may break in the future

Forgeo -> Forgejo

Forgeo -> Forgejo
JakobDev marked this conversation as resolved
@ -0,0 +29,4 @@
<img src="/assets/images/integrations/read-the-docs/admin-integrations.png" alt="webhooks">
</picture>
6. Now click on "Add Integregration" and select "GitHub incoming webhook" as type. You should see this Screen:

Integregration -> Integration

Integregration -> Integration
JakobDev marked this conversation as resolved

@JakobDev can you grant me push access to your fork? I'd like to make some spelling adjustments that are tedious to create a review for.

Does the integration work for private repos as well?

@JakobDev can you grant me push access to your fork? I'd like to make some spelling adjustments that are tedious to create a review for. Does the integration work for private repos as well?
Author
Contributor
Copy link

can you grant me push access to your fork?

I though you already have write access, because of I allow edit edits from maintainers in this PR, but you are now added.

Does the integration work for private repos as well?

That's a good question. I haven't tested it yet, but I don't think so. Read the Docs needs access to the Git Repo and is build for OpenSource Projects.

> can you grant me push access to your fork? I though you already have write access, because of I allow edit edits from maintainers in this PR, but you are now added. > Does the integration work for private repos as well? That's a good question. I haven't tested it yet, but I don't think so. Read the Docs needs access to the Git Repo and is build for OpenSource Projects.
@ -0,0 +10,4 @@
> Gitea/Forgejo is currenly not official supported by Read the Docs, so this may break in the future
## Create new Project
This steps shows how to create a new Project on Read the Docs
Contributor
Copy link

Thank you for this documentation!

I think you should add there that you need to have an account with readthedocs and that you must be signed in before being able to click on "Import a project" on the dashboad (https://readthedocs.org/dashboard/).

E.g. add a section:

## Registration
If you do not have an account yet, you must sign up with [readthedocs first](https://readthedocs.org/accounts/signup/).
After sign-up you need to login with your user to access your [dashboard](https://readthedocs.org/dashboard/).
Thank you for this documentation! I think you should add there that you need to have an account with readthedocs and that you must be signed in before being able to click on "Import a project" on the dashboad (https://readthedocs.org/dashboard/). E.g. add a section: ``` ## Registration If you do not have an account yet, you must sign up with [readthedocs first](https://readthedocs.org/accounts/signup/). After sign-up you need to login with your user to access your [dashboard](https://readthedocs.org/dashboard/).
JakobDev marked this conversation as resolved
@ -0,0 +12,4 @@
## Create new Project
This steps shows how to create a new Project on Read the Docs
1. On Read the Docs click "Import a Project"
Contributor
Copy link

I would rephrase it to:

1. On the [Read the Docs dashboard](https://readthedocs.org/dashboard/) click "Import a Project"
I would rephrase it to: ``` 1. On the [Read the Docs dashboard](https://readthedocs.org/dashboard/) click "Import a Project"
JakobDev marked this conversation as resolved
jklippel left a comment
Copy link

I accidentally approved before, so please see all of my comments as "request for change".

I accidentally approved before, so please see all of my comments as "request for change".
@ -0,0 +48,4 @@
<picture>
<source srcset="/assets/images/integrations/read-the-docs/add-webhook.webp" type="image/webp">
<img src="/assets/images/integrations/read-the-docs/add-webhook.png" alt="webhooks">
Contributor
Copy link

As already mentioned by @fnetX it would be great to have an alternate text for the images used.
E.g.:

instead of alt="webooks" one could use

alt="Image of the Forgejo 'Add Webhook' dialog. The input field for 'Target URL' is marked with a red border. Also the 'All Events' checkout below the 'Trigger On' section is marked with a red border."

That would help a lot.

As already mentioned by @fnetX it would be great to have an alternate text for the images used. E.g.: instead of alt="webooks" one could use ``` alt="Image of the Forgejo 'Add Webhook' dialog. The input field for 'Target URL' is marked with a red border. Also the 'All Events' checkout below the 'Trigger On' section is marked with a red border." ``` That would help a lot.
JakobDev marked this conversation as resolved
Ghost approved these changes 2023年02月19日 11:24:06 +01:00
@ -0,0 +25,4 @@
<picture>
<source srcset="/assets/images/integrations/read-the-docs/admin-integrations.webp" type="image/webp">
<img src="/assets/images/integrations/read-the-docs/admin-integrations.png" alt="webhooks">
Contributor
Copy link

Proposal for an alternative alt-attribute text:

Image of an admin website of a Read the docs project.
The integrations section of the admin page is selected.
The content of the Integrations sections states, that 'No integrations are currently configured.'.
An error message is shown in red saying that the project does not have a valid webhook set up.

Proposal for an alternative alt-attribute text: > Image of an admin website of a Read the docs project. > The integrations section of the admin page is selected. > The content of the Integrations sections states, that 'No integrations are currently configured.'. > An error message is shown in red saying that the project does not have a valid webhook set up.
JakobDev marked this conversation as resolved
@ -0,0 +32,4 @@
<picture>
<source srcset="/assets/images/integrations/read-the-docs/github-integration.webp" type="image/webp">
<img src="/assets/images/integrations/read-the-docs/github-integration.png" alt="webhooks">
Contributor
Copy link

Proposal for an alternative alt-attribute text:

Image of an admin website of a Read the docs project.
The integrations section of the admin page is selected.
The page content start with a header saying 'Integration - GitHub incoming webhook'.
After some explaining text an url specific to the Read the docs project is shown which can be used to manually configure this webhook.
The url shown is marked with a red border.

Proposal for an alternative alt-attribute text: > Image of an admin website of a Read the docs project. > The integrations section of the admin page is selected. > The page content start with a header saying 'Integration - GitHub incoming webhook'. > After some explaining text an url specific to the Read the docs project is shown which can be used to manually configure this webhook. > The url shown is marked with a red border.
JakobDev marked this conversation as resolved
@ -0,0 +40,4 @@
<picture>
<source srcset="/assets/images/integrations/read-the-docs/forgejo-webhook.webp" type="image/webp">
<img src="/assets/images/integrations/read-the-docs/forgejo-webhook.png" alt="webhooks">
Contributor
Copy link

Propsal for an alternative alt-attribute text:

Image of a Forgejo repository configuration.
The tab 'Webhooks' is selected.
The button 'Add Webhook' has been clicked and presents a list of possible Webhooks integrations.
The 'Forgejo' entry in the list has been marked with a red border.

Propsal for an alternative alt-attribute text: > Image of a Forgejo repository configuration. > The tab 'Webhooks' is selected. > The button 'Add Webhook' has been clicked and presents a list of possible Webhooks integrations. > The 'Forgejo' entry in the list has been marked with a red border.
JakobDev marked this conversation as resolved
jklippel left a comment
Copy link

Thanks for your contribution!

Thanks for your contribution!
JakobDev deleted branch readthedocs 2023年02月25日 15:19:19 +01:00
Sign in to join this conversation.
No reviewers
Labels
Clear labels
Codeberg Pages
Issues affecting Codeberg Pages
Documentation Usability
Issues related to using and reading docs.codeberg.org
Forgejo
Good First Issue! 👋
Kind: Bug
Kind: Documentation
Kind: Enhancement
Kind: Feature
Kind: Question
Kind: Security
Licensing
Part: Generator
This is related to the generation of the documentation, not to the content itself
Priority: High
The priority is high
Priority: Low
The priority is low
Priority: Medium
The priority is medium
Reviewed: Confirmed
Something has been confirmed
Reviewed: Duplicate
Something exists already
Reviewed: Invalid
Something was marked as invalid
Reviewed: Wontfix
Something won't be fixed
Status: Blocked
Status: Help wanted
Contributions are welcome!
Status: In progress
Work is in progress
Status: Needs feedback
Feedback is needed
Status: Ready for Review
Work is completed
Status: Review
Review is in progress / Reviewers wanted
Status: Stale
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Codeberg/Documentation!289
Reference in a new issue
Codeberg/Documentation
No description provided.
Delete branch ":readthedocs"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?