Peertube: Add link to instances

[bristow]

What happened?

In this page : admin/follows/followers-list, we have a list of instances that follow us but there is no link to consult this instance

What do you expect to happen instead?

I would like to have a link in order to see the content of these instances

Additional information

• PeerTube version or URL: 1.3.1
• Browser name/version: Firefox
• NodeJS version: ?

Thanks

Maybe we can put label 'Good for first issue' :)

[NassimBounouas]

Hi,

From what I see in my local instance, in the page admin/follows/followers-list we are listing all type of followers (peertube or mastodon). In this commit I wrapped the name displayed in a link pointing to the url stored on ActorModel.

Is it the behavior that you were expecting ?

[bristow]

I think you are right.

For example, i have [email protected] and i'd like to have a link (on this follower) in order to visit instance https://video.ploud.jp

Thanks @NassimBounouas :)

[NassimBounouas]

You're welcome @bristow, with this modification, in your example the link is to pointing to : https://video.ploud.jp/accounts/peertube/videos

I'll create a PR for this issue and the reviewer will tell us if something needs to be modified !

[bristow]

i'd prefer to go on https://video.ploud.jp/ (a lot of videos) instead of https://video.ploud.jp/accounts/peertube/videos (0 video) because this is for moderation (delete or not this follower).

[Chocobozzz]

You're welcome @bristow, with this modification, in your example the link is to pointing to : https://video.ploud.jp/accounts/peertube/videos

In your PR and the latest dev, using https://video.ploud.jp/accounts/peertube as URL will redirect to https://video.ploud.jp/videos/local so it should be fine.

[NassimBounouas]

Thank you @Chocobozzz for correction. I didn't see this behavior, where this redirection has been implemented ? (I didn't find it in the lastest commits)

[Chocobozzz]

@NassimBounouas https://github.com/Chocobozzz/PeerTube/commit/1a03bea0c42fa1064ce4770157b4fd2e3edd5565

Implemented by @NassimBounouas in https://github.com/Chocobozzz/PeerTube/commit/e1a1f9c623bfbbc7e38b36047926338f2e1c61d9

[bristow]

Thanks @NassimBounouas

I still have some issues in progress if you want to do your hand :)

https://github.com/Chocobozzz/PeerTube/issues/created_by/bristow

[bristow]

I saw the commit in action on https://peertube2.cpy.re/ and this is not really the expected need. Why not go directly to the root of the instance?

Because, if i take an example, i'm going here : https://video.ploud.jp/accounts/peertube/videos or here https://tube.govital.net/accounts/peertube/videos and there is no videos :(

Thanks

[Chocobozzz]

As I said here, when video.ploud.jp will be in 1.4 you'll be redirected to https://video.ploud.jp/videos/local.

We can't add a link that points to the instance root URL, because your instance is not necessarily followed by another peertube instance (it could be a mastodon account etc).

[bristow]

Sorry @Chocobozzz , i did not undestand it was for PT 1.4.

All is fine :+1: