Conversation
* Rename PeertubeUserExtractor to PeertubeAccountExtractor * Add test for video-channels in PeertubeChannelLinkHandlerFactoryTest * Compatibility support for older versions (use "accounts/" as default)
|
@Royosef: You forgot to address #308 (comment) |
I added the following test: Was I supposed to add something else? |
|
@Royosef: You was supposed to add that, but also tests for getting the URL from the ID. Those tests are for getting the ID from the URL. |
Oh, do you mean the acceptUrlTest function in PeertubeChannelLinkHandlerFactoryTest? I will add a test for video-channels there. |
|
@Royosef: No, I meant that you should add something like: Note that I didn't test whether this example works, and you should add all 3 cases. You're correct in that you also need to add a video channel to the |
add support for peertube channels