Skip to content

Conversation

@mbentley
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #494

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

@mbentley mbentley mentioned this pull request Jun 24, 2023
@netlify
Copy link

netlify bot commented Jun 24, 2023

Deploy Preview for homer-demo-content ready!

Name Link
🔨 Latest commit a842400
🔍 Latest deploy log https://app.netlify.com/sites/homer-demo-content/deploys/649703785a6c200008c5826e
😎 Deploy Preview https://deploy-preview-661--homer-demo-content.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@mbentley mbentley force-pushed the add-download-rate branch from e3fdfb5 to a842400 Compare June 24, 2023 14:53
@bastienwirtz bastienwirtz force-pushed the main branch 2 times, most recently from 12cad12 to 474dc3a Compare February 23, 2025 13:28
@mbentley mbentley force-pushed the add-download-rate branch from a842400 to 8f46a21 Compare April 5, 2025 13:59
@mbentley
Copy link
Contributor Author

mbentley commented Apr 5, 2025

Rebased against main just for good measure

@bastienwirtz
Copy link
Owner

Hey @mbentley, thanks a lot for your patience, sorry to be so late on this review.
The download rate has been added as a subtitle in the meantime, would that work for you ? The refresh interval and the dummy data from your PR would be great to have, If you are willing to rebase again and integrate it. I can do it if you prefer.
Thanks for your work @mbentley.

@mbentley
Copy link
Contributor Author

Subtitle's fine to me for the download rate. As for the updates, if you're able to do it, that's totally fine as I probably won't be able to get to it for a while.

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.

Sabnzbd Stats Support

2 participants