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

Add dedicated dotnet tab for dotnet portable build #1077

Merged
merged 3 commits into from
Sep 2, 2024

Conversation

gnattu
Copy link
Member

@gnattu gnattu commented Aug 14, 2024

We are listing these portable builds under all OS tabs, which may confuse users because each OS also has its own platform-specific builds that we refer to as “portable” in our documentation. To avoid confusion, separate the tabs accordingly.

This also updated the description as the .NET Core runtime is now just .NET runtime

We are listing these portable builds under all OS tabs, which may confuse users because each OS also has its own platform-specific builds that we refer to as “portable” in our documentation. To avoid confusion, separate the tabs accordingly.
@gnattu gnattu force-pushed the dedicated-dotnet-portable branch from 8746bde to 0269cf5 Compare August 14, 2024 23:45
@gnattu gnattu changed the title Add dedicated dotnet tab for dotnet protable build Add dedicated dotnet tab for dotnet portable build Aug 14, 2024
status: DownloadStatus.Official,
features: [],
platforms: [Platform.DotNet],
description: 'The portable version can be run on any system with a .NET Core runtime.',
description: 'The portable version can be run on any system with a .NET runtime.',
Copy link
Member

Choose a reason for hiding this comment

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

Good idea, though I think it should be even more explicit:

+description: 'The .NET portable version can be run on any system with a .NET runtime using dotnet jellyfin.dll.',

Copy link
Member Author

Choose a reason for hiding this comment

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

Well it seems like we cannot use markdown in this description, I change this to 'The .NET portable version can be run on any system with a .NET runtime using jellyfin.dll', which also explicitly states the user has to use the dll directly for this version.

@jellyfin-bot
Copy link

Cloudflare Pages deployment

Latest commit 8965d68359924b7ded12ee103c251db717471732
Status ✅ Deployed!
Preview URL https://aba28619.jellyfin-org.pages.dev
Type 🔀 Preview

@nielsvanvelzen nielsvanvelzen merged commit c5d1fd5 into jellyfin:master Sep 2, 2024
8 checks passed
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.

6 participants