search

ayrat555 / el_monitorro

🐂 El Monitorro is a high-performance feed reader as a Telegram bot. It supports RSS, Atom and JSON feeds
https://elmonitorro.badykov.com/
MIT License
201 stars 37 forks source link

fix: not to look title in order to insert a new feed #344

Open pxp9 opened 1 year ago

pxp9 commented 1 year ago

As requested of some users, the pr is to fix that el_monitorro sends an update when title is changed. This is an undesired behaviour.

pxp9 commented 1 year ago

https://github.com/ayrat555/el_monitorro/blob/master/src/sync/sync_feed_job.rs#L151-L155

This is the affected code

ayrat555 commented 1 year ago

@pxp9 there is already 

/set_content_fields url fields - changes content hash fields of the specified feed. it's available only for the admin (`ADMIN_TELEGRAM_ID`).
Example: /set_content_fields https://www.badykov.com/feed.xml author,title

maybe we can postpone these changes?

pxp9 commented 1 year ago

I think we should delete this pr if we are not adding this command for users.

That will take a lot of work.