tramitit / tramitit.github.io

Tramitit webpage
https://tramitit.com
MIT License
3 stars 0 forks source link

Fixing x-default in /guides #3

Closed rbatista191 closed 5 months ago

rbatista191 commented 5 months ago

Long thread, let me see if I can summarize it:

Screenshot 2024-06-11 at 12 46 54

@awennersteen can you help me on this?

Here is a good website to test the schema: https://technicalseo.com/tools/hreflang/

This is the official guide from Hugo for localization: https://gohugo.io/content-management/multilingual/

awennersteen commented 5 months ago

I think you had it correct, I made some changes whilst understanding but I think it was correct already: image

However I noted that if you check tramitit.com without pointing to a post it doesn't pass. I'll check if we can add something there

Edit: It appears that https://tramitit.com doesnt pass but https://tramitit.com/en does, I don't know if that is correct or not?

It also complains about duplicate references and such, I don't fully understand what these tags are for how they should be set so let me have a look

rbatista191 commented 5 months ago

Edit: It appears that https://tramitit.com doesnt pass but https://tramitit.com/en does, I don't know if that is correct or not?

It also complains about duplicate references and such, I don't fully understand what these tags are for how they should be set so let me have a look

@awennersteen the problem is the duplicate references indeed. I have an ongoing thread with Hugo's support, I also need to test these tags to see what matches what AHREF wants. But for the time being, I need to be able to remove the default ones.

See if my ticket to Hugo's support is clear, we can also have a call.

awennersteen commented 5 months ago

gotcha, now I understand the problem. Is the last reply by the guy the answer, looks like it to me, I'll try

rbatista191 commented 5 months ago

gotcha, now I understand the problem. Is the last reply by the guy the answer, looks like it to me, I'll try

Doesn't look like it, it still generates several rel=alternate:

Screenshot 2024-06-11 at 17 51 34
awennersteen commented 5 months ago

Yeah, so that fixed the case for english.

awennersteen commented 5 months ago

I still don't understand what's wrong with the alternate ones tbh

Edit: I now understand, I think I know how to debug

awennersteen commented 5 months ago

https://github.com/gethugothemes/hugo-modules/blob/3af75203b2efa73a0d98a29d4ca7beb1f7ec563d/seo-tools/basic-seo/layouts/partials/basic-seo.html#L31C1-L38C10

The same piece of code is in a base hugo module we use

rbatista191 commented 5 months ago

Issues fixed using the ongoing thread information. Thanks @awennersteen !