Differential D22072

Update links in aether
ClosedPublic
Actions

Authored by ognarb on Jun 24 2019, 2:12 PM.

Details

Reviewers
kde-www
jriddell
Summary

+ This remove the prefix of some links. Example https://kde.org/foo -> /foo
+ This remove the .php extension for some links. We should in the future redirect
with 301.
+ Add for the Reddit link https
+ Add for the mastodon link rel="me". This should verify the links in KDE Mastodon
account description.

Test Plan

Tested all the link locally

Diff Detail

Branch
/trunk/www/sites/www
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 13265
Build 13283: arc lint + arc unit
ognarb requested review of this revision.Jun 24 2019, 2:12 PM
ognarb created this revision.
ognarb added a reviewer: kde-www.Jun 24 2019, 2:13 PM
ognarb set the repository for this revision to R883 Subversion.
ognarb retitled this revision from Update link in aether to Update links in aether.Jun 24 2019, 2:15 PM
ognarb edited the summary of this revision. (Show Details)
jriddell accepted this revision.Jun 24 2019, 2:36 PM
This revision is now accepted and ready to land.Jun 24 2019, 2:36 PM
ognarb closed this revision.Jun 24 2019, 2:41 PM

Landed revision 1544955 in svn

aacid added a subscriber: aacid.Jun 24 2019, 4:16 PM
aacid added inline comments.
aether/footer.php
129

Can you explain why are you removing the .php part?

ognarb added inline comments.Jun 24 2019, 4:24 PM
aether/footer.php
129

Search engines prefer cleaner urls and for the users it's less information to read/copy/write. I would be in favor to redirect all the .php urls to the non php urls, but this will need some tests that nothing break.

Also I forgot to change this url. :/

Revision Contents
Changeset List

PathPackages
Maether/footer.php (36 lines)

Diff 60580

View Options

aether/footer.php

Loading...