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

[feature] Make RSS Feeds adhere to Notification settings #2131

Closed
meichthys opened this issue Jan 26, 2024 · 8 comments
Closed

[feature] Make RSS Feeds adhere to Notification settings #2131

meichthys opened this issue Jan 26, 2024 · 8 comments
Labels
enhancement New feature or request

Comments

@meichthys
Copy link

meichthys commented Jan 26, 2024

Thank you for this great tool!

Version and OS
0.45.13 on linux/docker

Is your feature request related to a problem? Please describe.
As a new user of changedetection I was expecting the content of the RSS feed to match the notification settings in changedetection. I was quite confused when i couldn't seem to change the Rss feed output to include the {watch_url} which is by default part of the notification output.

Describe the solution you'd like
I'd like to see RSS feeds adhere to the settings defined on the Notifications tab. Right now this is how my "Notification Body" is supposed to look like, but the setting seems to have no effect on the RSS feed, which is my preferred channel for reviewing changedetection updates:

{{watch_url}} had a change.
---
{{diff_added}}
---

Describe the use-case and give concrete real-world examples
When viewing an rss feed I would like to be able to click on a link to take me to the watch_url instead of the changedetection diff page.

Additional context
Resolving this issue would also resolve #327

@meichthys meichthys added the enhancement New feature or request label Jan 26, 2024
@dgtlmoon
Copy link
Owner

The problem is (i think i wrote elsewhere also), that the 'difference' is calculated when the notification is sent.

So what this means for RSS, is that basically every time you fetch the RSS feed, you might have 100 differences that need to be recalculated for each watch in the list

someone would have to check the performance impact there, maybe add some benchmark logging to the notification render/difference section of the code

@dgtlmoon
Copy link
Owner

This issue reads more or less a duplicate of #327

@meichthys
Copy link
Author

I would be totally fine without a diff in the notification. All I really was looking for is the watch_url to be included in the RSS feed.

@dgtlmoon
Copy link
Owner

I would be totally fine without a diff in the notification. All I really was looking for is the watch_url to be included in the RSS feed.

Oh snap, that is a bit annoying, that's an easy fix, i'll add that right away

@meichthys
Copy link
Author

Thanks!

dgtlmoon added a commit that referenced this issue Jan 29, 2024
@dgtlmoon
Copy link
Owner

@meichthys does this RSS template help? https://github.com/dgtlmoon/changedetection.io/pull/2139/files#diff-36cc0ceeaa435edb2f9fd2fd958397749fdd36cc4801b3c3a3c66fd28eae3be6R387

here it will provide a a href for that <h4> which will contain the watch title or URL (if title is not set)

@meichthys
Copy link
Author

meichthys commented Jan 29, 2024

Boom. YES! @dgtlmoon It would be nice to be able to configure that template myself, but that change would be a HUGE help for me. And now that i know where that template is in the codebase, i can play with it on my own 👍

@dgtlmoon
Copy link
Owner

Cool, this is good for now, next step will be to have a [RSS] tab in the settings I think, the RSS format i believe should be different to your notifications

(maybe you are watching with your RSS browser, but you also have email notification for some certain keyword setup etc)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants