search

thp / urlwatch

Watch (parts of) webpages and get notified when something changes via e-mail, on your phone or via other means. Highly configurable.
https://thp.io/2008/urlwatch/
Other
2.82k stars 352 forks source link

Add a csv2text filter #656

Closed Sveder closed 2 years ago

Sveder commented 3 years ago

Is there any interest in adding a csv2text filter? I'm sure this can be achieved with shellpipe and awk/sed magic, but I think an elegant way to do it is using a format string with the header or the column number. See basic implementation here: https://github.com/Sveder/urlwatch/pull/3/

@thp wdyt? Should I prepare a rebased pr?

thp commented 3 years ago

Yes, I think that would be a good addition. We have to make it use defaults and stuff and have some examples and maybe also expose the option to allow setting the delimiter, quoting, etc.. -- but if you want to work on that, please open a PR and let's continue the discussion there.

Sveder commented 3 years ago

I've added a rebase with my quick and dirty solution: https://github.com/thp/urlwatch/pull/658

Will wait till your comments on it. Thanks again for your time :)

thp commented 2 years ago

Closing this as, #658 is now merged. Thanks!