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 setting to ignore RSS rule matches for X days #802

Merged
merged 1 commit into from Nov 29, 2014

Conversation

Gelmir
Copy link
Contributor

@Gelmir Gelmir commented Jul 24, 2013

Partially implements: #223

Currently in test mode.

@sledgehammer999
Copy link
Member

Currently in test mode.

Does that mean that I should wait and not merge this?

@Gelmir
Copy link
Contributor Author

Gelmir commented Jul 25, 2013

Yep. Though I've tested this and haven't found any problems so far.

@sledgehammer999
Copy link
Member

Is this good to go?

@sledgehammer999 sledgehammer999 added this to the v3.2.0 milestone Nov 8, 2014
@Gelmir
Copy link
Contributor Author

Gelmir commented Nov 12, 2014

Should be ok to merge. Will rebase at weekend.

@sledgehammer999
Copy link
Member

Why rebase? github shows that this can be merged cleanly.

@Gelmir
Copy link
Contributor Author

Gelmir commented Nov 12, 2014

Just in case =)

2014-11-12 15:24 GMT+03:00 sledgehammer999 notifications@github.com:

Why rebase? github shows that this can be merged cleanly.


Reply to this email directly or view it on GitHub
#802 (comment)
.

@sledgehammer999
Copy link
Member

uhm ok. I'll wait for your notification.

@Gelmir
Copy link
Contributor Author

Gelmir commented Nov 12, 2014

k. Looks fine

2014-11-12 15:56 GMT+03:00 sledgehammer999 notifications@github.com:

uhm ok. I'll wait for your notification.


Reply to this email directly or view it on GitHub
#802 (comment)
.

@sledgehammer999
Copy link
Member

oops. It seems that by merging your other request, this one cannot be merged cleanly...

@sledgehammer999
Copy link
Member

@Gelmir A friendly reminder to rebase this, in case you didn't see my previous comment

@Gelmir
Copy link
Contributor Author

Gelmir commented Nov 25, 2014

@sledgehammer999 rebased to master

@sledgehammer999
Copy link
Member

Thank you.

sledgehammer999 added a commit that referenced this pull request Nov 29, 2014
Add setting to ignore RSS rule matches for X days
@sledgehammer999 sledgehammer999 merged commit b9ce4d4 into qbittorrent:master Nov 29, 2014
@Gelmir Gelmir deleted the rss_match_time_limit branch November 29, 2014 12:45
@nexxai
Copy link

nexxai commented Mar 9, 2015

This is probably a stupid question but where can I set this in the Windows client? I've looked through the Options / Advanced page, as well as in the RSS options and there aren't any settings that look like what this change added.

@sledgehammer999
Copy link
Member

It will be available in the coming v3.2.0 release. In the meantime you can use the latest alpha build: http://qbforums.shiki.hu/index.php/topic,2344.msg15534.html#msg15534

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.

None yet

3 participants