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

Can we get Commit 198ea6c merged #2789

Closed
4 tasks done
quadcom opened this issue Nov 12, 2024 · 2 comments
Closed
4 tasks done

Can we get Commit 198ea6c merged #2789

quadcom opened this issue Nov 12, 2024 · 2 comments

Comments

@quadcom
Copy link

quadcom commented Nov 12, 2024

Please complete the following tasks.

  • Web browser cache cleared
  • Link provided to install script if applicable
  • Not using broken rtinst install script
  • Web browser, ruTorrent, PHP and OS version provided

Tell us about your environment

FF/Ch
ExtRatio v 5.1 (as shown in the plugins list)

Can we get Commit 198ea6c reviewed and merged?

ExtRatio is broken, a least on my server and it appears that someone has already identified the problem and provided a fix on Oct 10th.

198ea6c

I could not find any ticket regarding this problem yet.

https://youtu.be/YqZyCBziI2I

Tell us how you installed ruTorrent

Seedbox provider

Describe the bug

ExtRatio is not applying the config created in the UI.

Steps to reproduce

Clear your Ratio Rules list
reiew the blank ratiorules.dat file.
Add a rule via the UI.
Compare the default ratiorules.dat with the new version that has the new rule.
The new rule in the ratiorules.dat file will not match the config entered in the UI.
The default mininova.org match is saved in the dat.

Expected behavior

No response

Additional context

No response

@quadcom quadcom changed the title Can we get Commit 198ea6c pushed in..... Can we get Commit 198ea6c merged Nov 12, 2024
@stickz
Copy link
Collaborator

stickz commented Nov 13, 2024

Thanks for the youtube video @quadcom ! Whatbox is running a stable - bleeding edge configuration of ruTorrent. v5.1 beta 4. Therefore, the mentioned commit is already installed on your server. Merged as part of #2725.

ExtRatio is a new bleeding edge plugin, that was not previously possible to enable on Whatbox due to upstream limits. We removed this limitation in v5.1. We will investigate and resolve any associated problems with this new feature!

Please let us know if Beta 5 resolves this problem, once it gets deployed to Whatbox. We have 1 more issue to resolve, then we'll be tagging a newer release control version for @anthonyryan1 at Whatbox to deploy.

@quadcom
Copy link
Author

quadcom commented Nov 14, 2024

@stickz my issue was resolved with your fix. This issue may not be related after all.

@quadcom quadcom closed this as completed Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants