search

jygastaud / hugo-microtypo

That project is a try to build a Hugo module by porting [Jekyll Microtypo](https://github.com/borisschapira/jekyll-microtypo)
8 stars 4 forks source link

fix: duplicate content with a lang different from "fr" #12

Closed bgatellier closed 1 year ago

bgatellier commented 1 year ago

Hi @jygastaud

I was struggling with an error of duplicate content when trying to translate the ecoinde.fr website into other languages than "fr". It seems that doing the changes in this PR fix the problem (see my fork here) However, as I just discovered Hugo and Go with the ecoindex.fr projet, I'm not completly sure that this fix is the good way to solve the issue. So if you have a better suggestion, feel free to do it instead :)

jygastaud commented 1 year ago

Hi,

As discussed with @yaaax on Slack, I think that bug happen with recent version of Hugo as I can't see it on my blog website which still use an old version of Hugo.

Thinking about the different solution, I'm OK that the best solution is just to remove the else condition which is not useful.

So, I will accept you PR.

jygastaud commented 1 year ago

And, of course, thank you @bgatellier for your contribution.