jamulussoftware / jamuluswebsite

This is the GitHub Pages repository for the Jamulus main website. For the jamulus application source code, please visit jamulussoftware/jamulus.
https://jamulus.io
GNU Lesser General Public License v2.1
16 stars 79 forks source link

Add canonical links for Google #988

Open gilgongo opened 3 weeks ago

gilgongo commented 3 weeks ago

==> UPDATE <===

These changes are probably better incorporated with @ignotus666 changes to the language selector. So putting in draft for now.

Short description of changes

  1. Fix for https://github.com/jamulussoftware/jamuluswebsite/issues/986
  2. Adds fully-qualified links to KB articles in sitemap.xml which I think might be responsible for "invalid URL" errors from Google.

What is missing until this pull request can be merged? Ideally, need to work out how to get _includes/general/nav.html to work, but it's not doing it when the current language is en for some reason.

BTW I'm not 100% sure that adding rel=canonical to the EN version of links is necessarily going to fix Google's problem, since they imply we can only "suggest" the canonical versions, but hopefully this will make things a bit better.

Does this need translation?

No

Checklist

gilgongo commented 3 weeks ago

BTW the only way I can get the site to work without errors after cloning it locally (on Ubuntu 24.04) is to:

bundle add webrick then change to liquid (4.0.4) in Gemfile.lock (Also had to use po4a v0.69 but @ignotus666 gave me v0.72 which also works)

ignotus666 commented 3 weeks ago

I had a go yesterday and surprisingly enough didn't run into any problems - the usual commands worked. I'm on Mint 21.3 (based on Ubuntu 22.04).

gilgongo commented 3 weeks ago

Ok so maybe it's some ruby version dependency thing. I'll see if I can reproduce it on a fresh 24.04 installation and maybe add a note to the docs since the error messages are pretty unhelpful.

On Sun, 9 Jun 2024, 12:34 pm ignotus, @.***> wrote:

I had a go yesterday and surprisingly enough didn't run into any problems

  • the usual commands worked. I'm on Mint 21.3 (based on Ubuntu 22.04).

— Reply to this email directly, view it on GitHub https://github.com/jamulussoftware/jamuluswebsite/pull/988#issuecomment-2156455472, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCZWU4V3EZQDBTZRGVG3WDZGQ4U3AVCNFSM6AAAAABI4NM4ICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGQ2TKNBXGI . You are receiving this because you authored the thread.Message ID: @.***>

gilgongo commented 2 weeks ago

I think we may be best merging this after the language code fix is merged, since Google might treat rel=alternate differently if it also knows the URL is in a given language.