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

feat(docs): Changes to the Administration section. #8078

Merged
merged 5 commits into from
Mar 24, 2024

Conversation

aviv926
Copy link
Contributor

@aviv926 aviv926 commented Mar 19, 2024

This PR adds information about the system settings, although most of the information already exists within the Immich browser interface, I saw fit to document it and add more information as I saw fit.

In addition, I deleted the password-login.md page because:

  1. It is mentioned on the new page together with all the system settings.
  2. The FAQ mentions how an administrator can reset his password.

I made appropriate changes to the _redirects page that will be sent in a separate PR because it contains references to pages that will be deleted in additional PRs that I will send soon.

In addition, a short explanation was added to the Jobs page.

@aviv926 aviv926 added the documentation Improvements or additions to documentation label Mar 19, 2024
@aviv926 aviv926 changed the title Administration feat(docs): Changes to the Administration section. Mar 19, 2024
Copy link
Contributor

@jrasm91 jrasm91 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks!

@@ -24,3 +24,4 @@
/docs/features/user-management /docs/administration/user-management 301
/docs/developer/contributing /docs/developer/pr-checklist 301
/docs/guides/machine-learning /docs/guides/remote-machine-learning 301
/docs/administration/password-login /docs/administration/system-settings 301
Copy link
Contributor

Choose a reason for hiding this comment

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

Reorganizing content is great, but try to remember to add a record here so that we don't end up with broken links due to those changes 🙏

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, I just read your comment in the description about this. If we have separate PRs for different sections, just include the relevant line immediately. Otherwise, as soon as the PR is merged, the links will be broken.

@jrasm91 jrasm91 enabled auto-merge (squash) March 24, 2024 02:03
@jrasm91 jrasm91 merged commit d36d32d into immich-app:main Mar 24, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants