[feature] Notification includes text, that is in "Ignore Text"

[chris2k20]

Version and OS v0.40.1.1

Is your feature request related to a problem? Please describe. Notification includes Text, that is in the "Ignore Text" list.

Describe the solution you'd like The regex in the "Ignore Text" is successfully not triggering a change, but it should be also ignored in the notification itself, because I got the ignored text (when it is near a non-ignore text), specially I am using that filter:

/.*watched.*/
/.*comments.*/

Describe the use-case and give concrete real-world examples Keep the notification so short as possible and filter out useless information.

Additional context That is included in my telegram notification:

(added )    6 people watched this today
(added )    XYZ

[dgtlmoon]

nice suggestion!

[N1ghtshad0w]

I am having the same issue and been trying to find a fix or a workaround for a while now with no luck.

Version and OS v0.40.3

Is your feature request related to a problem? Please describe. Notification includes Text, that is in the "Ignore Text" list.

Describe the solution you'd like The word in the "Ignore Text" is successfully not triggering a line change, but it should be also ignored in the notification itself, because I got the ignored line, specially I am using that filter:

ago

Describe the use-case and give concrete real-world examples Keep the notification so short as possible and filter out useless information.

Additional context That is included in my telegram notification that makes finding the actual change in the middle of all this clutter really annoying:

(changed)           11 minutes ago
(into   )           2 hours ago
(changed)           3 hours ago
(into   )           5 hours ago
(changed)           13 hours ago
(into   )           14 hours ago
(changed)           19 hours ago
(into   )           21 hours ago
(changed)           23 hours ago
(into   )           1 day ago
(changed)           3 days ago
(into   )           4 days ago

[chris2k20]

Yeah, good point! Is somebody already working on a PR to fix that?

[dgtlmoon]

I think its best to use a new token {{diff_clean}} which would contain everything that is not in ignore perhaps ? yes no? I dont want to change the behaviour of that existing token for the other billion people using this app

Naming ideas..

• diff_clean ?
• diff_without_ignore ?

[chris2k20]

Having a diff_without_ignore would be great :)

[N1ghtshad0w]

Having a diff_without_ignore would be great :)

I second that 👍

[chris2k20]

Is there any update because I really need that feature?