search

libertysoft3 / saidit

The reddit open source fork powering SaidIt
https://saidit.net
Other
228 stars 40 forks source link

Add Invidious support #41

Closed wizzwizz4 closed 5 years ago

wizzwizz4 commented 5 years ago

Please, please, please test this before merging. I think I got everything, but I'm not sure.

wizzwizz4 commented 5 years ago

39 and I think I missed something.

libertysoft3 commented 5 years ago

Please, please, please test this before merging.

Installing Ubuntu 14 in a VM and getting saidit going is not that bad. Not that I'm gonna be a beggy chooser and turn away helpful code.

wizzwizz4 commented 5 years ago

We should be writing unit tests, shouldn't we?

libertysoft3 commented 5 years ago

Yeah probably. There are some tests in there, I have been avoiding them.

libertysoft3 commented 5 years ago

Thanks wizzwizz

libertysoft3 commented 5 years ago

Here's what you were missing, not too bad for a no-test: https://github.com/libertysoft3/saidit/commit/7011036bdf2d60fbfdba6bcc7f73b6ec548ef1f0