search

ckampfe / russ

A TUI RSS reader with vim-like controls and a local-first, offline-first focus
GNU Affero General Public License v3.0
173 stars 18 forks source link

Delete feed feature #3

Closed Funami580 closed 2 years ago

Funami580 commented 3 years ago

Something like this?

ckampfe commented 2 years ago

I will try to look at this soon, thanks again for your contributions.

ckampfe commented 2 years ago

Sorry for the delayed review. Thank you again for the contribution @Funami580. Overall I think I like this. I personally have not needed to delete feeds that often, and when I have, I usually just go into the database and do it with sql, but I believe this will be very helpful for users who are not as comfortable with sql.

If you wouldn't mind adapting the one commented section to occur in a single transaction, I will test and merge. Thank you!

Funami580 commented 2 years ago

Ok, done. Should be ready to test.

ckampfe commented 2 years ago

@Funami580 this is great, thank you again for your contributions, merging now