-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Shorter URLs channel and account + /@:actorName #4009
Shorter URLs channel and account + /@:actorName #4009
Conversation
ad9697a
to
0cc221a
Compare
10a83a2
to
7df8e14
Compare
7df8e14
to
02ffda7
Compare
Since the last commits now we can use :
Server side, a new API route is used for actors. This way we could implement an URL even shorter for videos : You can see more details in the updated description of this PR. @Chocobozzz, did you have this feature implemented like that in mind? If it looks good to you, the issue #4011 might be resolved easily for channels after merging this one. |
02ffda7
to
0c37efc
Compare
5bb2827
to
3cdb0b2
Compare
59b1663
to
030ccfc
Compare
@Chocobozzz @rigelk is there something you wait for this PR ? something I can do ? I thought it was related to UI/UX of channels / accounts profiles, do you plan to merge it after the 3.2 ? |
Yes, I'll update and merge the PR for 3.3 because the resumable upload PR took a lot of my time. |
Thanks no pb ! I saw this huge PR, it's good to have releases size well-balanced. |
Description
Creates shorter URLs for channels and accounts.
Client side
/a/:accountName
for accounts and/c/:accountName
for channels/@:actorName
displays:/a/:accountName
or/c/:channelName
without the tab home (exemple:/c/mychannel/videos
)/accounts/:accountName
to/a/:accountName
/video-channels/:channelName
to/c/:channelName
Server side
/accounts/:accountName
for/a/:accountName
and/@accountName
/video-channels/:channelName
for/c/:channelName
and@channelName
GET /api/v1/actors/:actorName
to handle/@:actorName
client route + testRelated issues
Partially resolves #3680
Has this been tested?