Skip to content

Conversation

marcokreeft87
Copy link
Contributor

@marcokreeft87 marcokreeft87 commented Mar 30, 2025

Description

Added 2 services, Plex and docker proxy service.
Added down and upload rate to Sabnzbd service

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • I've read & comply with the contributing guidelines
  • I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.
  • I have made corresponding changes to the documentation (README.md).
  • I've checked my modifications for any breaking changes, especially in the config.yml file

@marcokreeft87 marcokreeft87 reopened this Mar 30, 2025
Copy link
Owner

@bastienwirtz bastienwirtz left a comment

Choose a reason for hiding this comment

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

Nice work @marcokreeft87 !
Thanks you for this great addition. A few detail and we should be good to go !

<p class="title is-4">{{ item.name }}</p>
<p class="subtitle is-6">
<span v-if="error" class="error">An error has occurred.</span>
<template v-else>
Copy link
Owner

Choose a reason for hiding this comment

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

Could you also handle the subtitle ? Other services usually works like this : if a subtitle is defined, then display it, otherwise, display the extra information (this way there is still both possibility)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Isn't that handled by the templating of generic component?

Copy link
Owner

Choose a reason for hiding this comment

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

Once you override the #content template, the content provided by the generic component is not applied anymore.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My latest commit handles this

@marcokreeft87
Copy link
Contributor Author

Nice work @marcokreeft87 ! Thanks you for this great addition. A few detail and we should be good to go !

I've change everything you've asked, I think haha. Thnx for the suggested improvement :)

Copy link
Owner

@bastienwirtz bastienwirtz left a comment

Choose a reason for hiding this comment

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

Thanks for your modification just one last question, and we should be good to go !


// For upload speed, you would need a corresponding field in the API response.
// Assuming there's no direct upload speed in the response, you can modify as needed.
this.ulSpeed = 0; // Placeholder for upload speed (API might need to provide this)
Copy link
Owner

Choose a reason for hiding this comment

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

So the upload speed is not available ?
Out of curiosity, is the comments / code from an LLM ? 😃

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes the comments are from ChatGPT. But good spot. SabNzbd can't upload :D Didn't really notice that, I've copied the idea from the qBittorrent service, thats why the upload is there

@bastienwirtz bastienwirtz merged commit b40d008 into bastienwirtz:main Apr 14, 2025
5 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.

2 participants