deleting feeds with a trailing slash

Question

deleting feeds with a trailing slash

localguru opened this issue 7 years ago · comments

Hi Robert,

I had some problems today to remove an URL which I added with a trailing slash. RocketChat doesn't display a trailing slash and rss list doesn't show it. So if you want to delete the URL later and don't remember the trailing slash the URL is not found ;)

See here: RocketChat/Rocket.Chat#7527

I would classify this as a RocketChat problem, but if deleting an URL a workaround could be to test both versions of an URL with and without a trailing slash.

Ciao!

Robert Müller · Answer 1 · Tue Aug 29 2017 01:40:23 GMT+0800 (China Standard Time)

@localguru I've fixed the check and it should now delete both with and without the trailing slash.

The fix is published with version 1.1.1 to npm

Due to the lack of tests please let me know if this works 🤔

Also I'am very new to coffeescript so this might not the best fix. PRs welcome

Marcus Schopen · Answer 2 · Tue Aug 29 2017 03:21:58 GMT+0800 (China Standard Time)

@Flipez Thanks, Robert! 👍 I will test and give you feedback.

Marcus Schopen · Answer 3 · Tue Aug 29 2017 03:41:10 GMT+0800 (China Standard Time)

It's working perfectly, nice! 🥇