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
Move Ignore Origin rules to database #4026
Conversation
Why the need for RuleInterface? What do tagging rules and these origin rules have in common ? |
@tcitworld they have Config and Id getters and setters in common. Making an interface for these two entities makes me reuse |
1751fbb
to
143abe9
Compare
b1ab2ea
to
4a40fe8
Compare
4a40fe8
to
dd20c74
Compare
dd20c74
to
9e5a1f7
Compare
347ff08
to
6445179
Compare
6445179
to
254810f
Compare
254810f
to
695a9a8
Compare
695a9a8
to
a3c6ff4
Compare
This PR is finally ready for review, enjoy @wallabag/core |
361a189
to
125ec65
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok to me 👍
I will try the feature before merging it, ok? |
@nicosomb yeah sure, go on |
125ec65
to
84e316a
Compare
84e316a
to
773625e
Compare
@nicosomb did you test it? |
Yes. As I said on gitter, I think that explanations are not so clear but I don't see how we can improve them. |
Add IgnoreOriginUserRule for user-defined rules and IgnoreOriginInstanceRule for system-wide rules. Add an interface for these two new entities. Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
This commits adds a new helper like RuleBasedTagger for processing ignore origin rules. It also adds a new custom RulerZ operator for the '~' pattern matching rule. Renames 'pattern' with '_all' in IgnoreOriginRule entity. Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
Excluding this group can decrease the run time of tests during development. Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
773625e
to
71f7e58
Compare
Fixes #3760