Skip to content
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

revamp docs: improved discoverability of common help topics #16392

Closed
wants to merge 14 commits into from

Conversation

noerw
Copy link
Member

@noerw noerw commented Jul 10, 2021

This PR is a general sweep over the docs to (a) bring them up to date, (b) make them more helpful

reorder content in a way that docs show the most common topics and problems at the top, to improve helpfulness of the docs

  • categorize FAQ and reorder questions
  • fix categorization of pages
    • add new overview, configuration categories
    • dissolve upgrade category (move upgrade from gogs to installation)
    • dissolve features category
      • localization: →translation
      • comparison: →overview
      • authentication: →usage
      • webhooks: →usage
    • clean up usage category (plenty of pages that should be in configuration)
    • clean up advanced category (plentry of pages that should be in usage or configuration)
    • integrations: developers→overview

consolidate info (avoid duplication between multiple pages)

  • custom themes
  • ssh issues
  • TODO check more pages...

fix minor errata, update docs content, provide more detail

@noerw noerw added the type/docs This PR mainly updates/creates documentation label Jul 10, 2021
@codecov-commenter
Copy link

codecov-commenter commented Jul 10, 2021

Codecov Report

Merging #16392 (cb6efea) into main (0393a57) will increase coverage by 1.26%.
The diff coverage is 52.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #16392      +/-   ##
==========================================
+ Coverage   44.24%   45.50%   +1.26%     
==========================================
  Files         695      709      +14     
  Lines       82341    83767    +1426     
==========================================
+ Hits        36429    38121    +1692     
+ Misses      40014    39511     -503     
- Partials     5898     6135     +237     
Impacted Files Coverage Δ
cmd/convert.go 0.00% <0.00%> (ø)
cmd/dump.go 0.91% <0.00%> (ø)
cmd/dump_repo.go 0.00% <0.00%> (ø)
cmd/generate.go 0.00% <0.00%> (ø)
cmd/hook.go 0.00% <0.00%> (ø)
cmd/migrate.go 0.00% <0.00%> (ø)
cmd/migrate_storage.go 0.00% <0.00%> (ø)
cmd/serv.go 2.36% <0.00%> (-0.22%) ⬇️
cmd/web.go 0.00% <0.00%> (ø)
cmd/web_letsencrypt.go 0.00% <ø> (ø)
... and 281 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a40206...cb6efea. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 10, 2021
@noerw noerw marked this pull request as draft July 10, 2021 13:38
@noerw noerw added the pr/wip This PR is not ready for review label Jul 10, 2021
@noerw noerw changed the title docs: update url to external repo for STL renderer revamp docs: improved discoverability of common help topics Jul 10, 2021
[Swagger](https://swagger.io/) is what Gitea uses for its API.
All Gitea instances have the built-in API, though it can be disabled by setting `ENABLE_SWAGGER` to `false` in the `api` section of your `app.ini`
For more information, refer to Gitea's [API docs]({{< relref "doc/developers/api-usage.en-us.md" >}})
See [here]({{< relref "doc/advanced/customizing-gitea.en-us.md" >}}#customizing-the-look-of-gitea).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally best to avoid here links:

Suggested change
See [here]({{< relref "doc/advanced/customizing-gitea.en-us.md" >}}#customizing-the-look-of-gitea).
See the [customizing the look of gitea]({{< relref "doc/advanced/customizing-gitea.en-us.md" >}}#customizing-the-look-of-gitea) section of our documentation.

Comment on lines +251 to +253
The most common culprit for this is loading federated avatars.
This can be turned off by setting `ENABLE_FEDERATED_AVATAR` to `false` in your `app.ini`
Another option that may need to be changed is setting `DISABLE_GRAVATAR` to `true` in your `app.ini`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So we solved this particular issue when we pushed the federated avatar lookup to when the avatar is requested.


The usual cause of slowdowns right now is the last commit lookup - hopefully #16467 and #16432 will fix that one - and then it will be the diff renderer which I might be able to get to if those would get reviewed and merged.

There is no setting for password resets. It is enabled when a [mail service]({{< relref "doc/usage/email-setup.en-us.md" >}}) is configured, and disabled otherwise.
### Issues after updating

Please always check the release notes / [changelog](https://github.com/go-gitea/gitea/blob/main/CHANGELOG.md) for breaking changes; we regularly have breaking changes on minor releases!
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We really try not to have Breaking changes on minor releases...

There are breaking changes on 1.x.0 releases - we very much try not to make breaking changes on 1.x.y releases.

The main suggestion though should be to always jump to the latest minor release - that is go from:

1.13.3 -> 1.13.7 -> 1.14.5 NOT 1.13.3 -> 1.13.4 -> 1.13.5 -> ... -> 1.14.0 -> 1.14.1 ... -> 1.14.5

@wxiaoguang
Copy link
Contributor

Is it active? The documents have been hugely changed recently.

@wxiaoguang wxiaoguang added the issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail label Apr 24, 2023
@wxiaoguang
Copy link
Contributor

It has been stale for a long time and I can't think of a way to handling it other than closing it. Feel free to reopen if there's any new progress and I could also help.

@wxiaoguang wxiaoguang closed this May 1, 2023
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Aug 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. pr/wip This PR is not ready for review type/docs This PR mainly updates/creates documentation
Projects
None yet
5 participants