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

added contains/not-contains filter types #763

Merged
merged 1 commit into from
Aug 24, 2024

Conversation

LeXofLeviafan
Copy link
Collaborator

tag name search

tag contains

bookmark URL search

url contains
url not contains

bookmark title search

title contains
title not contains

bs_filters.BookmarkBaseFilter(name, filter_type=FilterType.SMALLER),
bs_filters.BookmarkBaseFilter(name, filter_type=FilterType.TOP_X),
bs_filters.BookmarkBaseFilter(name, filter_type=FilterType.BOTTOM_X),
]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Also reformatted filter declarations a bit

bs_filters.BookmarkBaseFilter(name, filter_type=FilterType.NOT_IN_LIST),
bs_filters.BookmarkBaseFilter(name, filter_type=FilterType.CONTAINS),
bs_filters.BookmarkBaseFilter(name, filter_type=FilterType.NOT_CONTAINS),
]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The filters were added at the end of their respective lists (just in case)

@jarun jarun merged commit 314861a into jarun:master Aug 24, 2024
1 check passed
@jarun
Copy link
Owner

jarun commented Aug 24, 2024

Thank you!

@LeXofLeviafan LeXofLeviafan deleted the filter-contains branch August 24, 2024 17:56
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