Similar to #3034235: Use alias redirection target when creating goto/login link, I think users' intentions are clearly for redirect targets to be the canonical domain for a site. This should be reflected in the title displayed on site nodes, views, etc.

Comments

ergonlogic created an issue. See original summary.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Project: Hostmaster (Aegir) » Hosting

This is a UI change, so it should be limited to hosting.module... Maybe Eldir too, but let's start here.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Related issues: +#3034235: Use alias redirection target when creating goto/login link
ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented

Here's a first pass at this: https://cgit.drupalcode.org/hosting/commit/?h=dev/consensus&id=987201ba4...

I think all that's left is to add a Views plugin to alter the titles displayed in our listings of sites.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Status: Active » Needs review

Here are the Views handlers, and updated default views: https://cgit.drupalcode.org/hosting/commit/?h=dev/consensus&id=a93bb76bc...

I believe this is now complete.

Since this is a change in the UX, we could at a settings checkbox to keep the current behaviour by default. That said, it only affects sites that are using redirects, so I don't know how wide-spread the impact will be.

llamech’s picture

llamech
He/him
commented

Fixes for this, along with other subdirectory site fixes, are all in #3036890: Simplify subdir site installation. For simplicity, let's validate there.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Status: Needs review » Fixed

We're merging fixes from #3036890: Simplify subdir site installation, which resolves this issue too.

  • llamech committed 74d0c18 on 7.x-3.x authored by ergonlogic 
    Issue #3036884: Initial pass at using redirect target for site title in...
  • llamech committed 81db15f on 7.x-3.x authored by ergonlogic
    Issue #3036884: Add Views handlers to allow canonical URLs.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented
Parent issue: » #3122928: [META] Support subsites / subdirectory sites like `example.com/foo` (Part III)