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

Docs: Installing on TrueNAS #738

Closed
wants to merge 19 commits into from
Closed

Docs: Installing on TrueNAS #738

wants to merge 19 commits into from

Conversation

DjP-iX
Copy link
Contributor

@DjP-iX DjP-iX commented Nov 6, 2023

This PR introduces documentation on installing the Jellyfin application in TrueNAS SCALE.

Note: I've left the sidebar_position undefined to allow you to weight as needed.

docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
@DjP-iX
Copy link
Contributor Author

DjP-iX commented Dec 18, 2023

ty for the suggestions @felix920506! Might take me a few weeks to circle back to this, but I will be reworking to address them

@DjP-iX DjP-iX requested a review from felix920506 January 18, 2024 17:47
Copy link
Member

@felix920506 felix920506 left a comment

Choose a reason for hiding this comment

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

Reading through this guide, I find quite a bit of duplicate information on the bottom. I don't know if this is intended or not. However, I find the info on the bottom more useful when trying to configure the App compared to their counterparts on the top.

There are also a few things, please check my comments in the middle.

docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
@felix920506
Copy link
Member

@DjP-iX is this ready for re-review?

@DjP-iX
Copy link
Contributor Author

DjP-iX commented Apr 15, 2024

I haven't looked at it in some time, but as far as I know it is

docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
docs/general/installation/truenas.md Outdated Show resolved Hide resolved
Copy link
Contributor

@Gauvino Gauvino left a comment

Choose a reason for hiding this comment

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

LGTM !

@joshuaboniface
Copy link
Member

Superseded by #1228 and credited as coauthor in merge commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants