-
Notifications
You must be signed in to change notification settings - Fork 23
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 browser activity filter #234
Conversation
Let's rename to |
I don't like the |
Not sure I understand exactly what you mean... |
This is renamed now. Let me know what to do about the @@ syntax then I will finalize this PR. |
For For now, let's just not support the |
Also, we need a setting to toggle sending |
Actually, let's make the allow/deny list regexes which can match the full url. |
7d475c0
to
f297e8d
Compare
I implemented that and removed the @@ syntax. Do you want to add something to the remarks below the text view? |
Thank you! That's so much better than |
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.
🚀 Devs have waited for a long time for this feature... you've made them very happy!
I took the liberty to redesign the Settings window a bit:
The whitelist also supports the
@@
syntax.Closes #196