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

Post-processing script #29

Closed
1 task done
mntn-xyz opened this issue Jan 28, 2024 · 1 comment
Closed
1 task done

Post-processing script #29

mntn-xyz opened this issue Jan 28, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@mntn-xyz
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Feature description

This came to me after thinking more about #28 and realizing that this could be a better way to add this and similar features without creating bloat.

It would be helpful if russ had the option to run a post-processing script after items are loaded from a feed. Ideally the script would be passed a list of database IDs (as a list of arguments) for the new feed items so it can do whatever needs to be done for those items, in whatever order is best.

Some potential use cases:

  • Fetching HTML article contents and parsing them into text
  • Fetching article images and storing them somewhere
  • Running filters to remove articles that match or don't match certain keywords/tags

It would be better to set this script at the individual feed level, as this would allow users to easily set different post-processors for different feeds.

I'm not sure whether or not russ should wait for the script to complete, and how long it should wait. For simplicity's sake it may be best to just wait with a spinner or something to indicate activity, with the user able to press some key to stop waiting. If and when the script finishes, the user would see success or failure status.

@mntn-xyz mntn-xyz added the enhancement New feature or request label Jan 28, 2024
@ckampfe
Copy link
Owner

ckampfe commented Jun 1, 2024

This is a very interesting idea but I'm not interested in adding a scripting interface to russ at this time. I do want to thank you both for opening the issue and for typing up an exploration of this idea. Scripting may be something I revisit in the future.

@ckampfe ckampfe closed this as completed Jun 1, 2024
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