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): New repair and statistics pages #8030

Merged
merged 6 commits into from
Mar 23, 2024
Merged

Conversation

aviv926
Copy link
Contributor

@aviv926 aviv926 commented Mar 17, 2024

This PR adds information to the documentation on the repair page and the statistics page. In addition, I added some tips that may be useful.

@aviv926 aviv926 added the documentation Improvements or additions to documentation label Mar 17, 2024
@aviv926 aviv926 changed the title feat(docs):New repair and statistics page feat(docs): New repair and statistics page Mar 17, 2024
@aviv926
Copy link
Contributor Author

aviv926 commented Mar 17, 2024

It failed due to a future change to the file
remove-offline-files.md which will become remove-offline-paths.md
To specify the purpose of the page.

docs/docs/administration/Repair-page.md Outdated Show resolved Hide resolved
docs/docs/administration/Repair-page.md Outdated Show resolved Hide resolved
Several users report a situation where the page fails to load, in order to solve this problem you should try to allocate more RAM for Immich, if the problem continues, you should stop using the reverse proxy while loading the page.
:::

Any other state describes the following 3 states:
Copy link
Member

Choose a reason for hiding this comment

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

This sentence doesn't make sense to me.

Copy link
Member

Choose a reason for hiding this comment

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

I still don't understand what you're trying to say with this @aviv926

Copy link
Member

Choose a reason for hiding this comment

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

Ah thanks!

docs/docs/administration/Repair-page.md Outdated Show resolved Hide resolved
docs/docs/administration/Repair-page.md Outdated Show resolved Hide resolved
@aviv926
Copy link
Contributor Author

aviv926 commented Mar 18, 2024

Thanks for the feedback, corrected accordingly.
By the way, regarding the screenshots, I believe I will update them with the new logo in a future PR, but it is less critical for now.

@aviv926 aviv926 requested a review from danieldietzler March 18, 2024 22:49
@aviv926 aviv926 changed the title feat(docs): New repair and statistics page feat(docs): New repair and statistics pages Mar 18, 2024
@aviv926 aviv926 requested a review from alextran1502 March 19, 2024 14:46
Copy link
Member

@danieldietzler danieldietzler left a comment

Choose a reason for hiding this comment

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

Thank you! And sorry for not understanding that one line lol

@jrasm91 jrasm91 enabled auto-merge (squash) March 22, 2024 23:21
@jrasm91 jrasm91 merged commit 3cc800f into immich-app:main Mar 23, 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.

4 participants