Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unnecessary redirects for multilingual pages with user-friendly URLs #731

Closed
4 tasks done
fredericalpers opened this issue Jan 23, 2024 · 7 comments · Fixed by #740
Closed
4 tasks done

Unnecessary redirects for multilingual pages with user-friendly URLs #731

fredericalpers opened this issue Jan 23, 2024 · 7 comments · Fixed by #740
Assignees
Labels
bug Indicates an unexpected problem or unintended behaviour component: dashboard Issue, Pull Request or Discussion related to dashboard / settings page component: property detail page Issue, Pull Request or Discussion related to property detail page QA Issue or Pull request that is in review
Milestone

Comments

@fredericalpers
Copy link
Member

fredericalpers commented Jan 23, 2024

Initiative

Multilingual SEO optimization

Epic

URL structure enhancement for language switch

User story

As a real estate agent aiming for optimal SEO ranking and user-friendly URLs, I want the URL redirection during language switches to be seamless and consistent. Currently, the language switch causes unnecessary redirections and inconsistent URL structures, potentially impacting the SEO ranking.

The current implementation of URL redirection during a language switch results in suboptimal outcomes. Transitioning from German to English involves an interim redirection to a URL without the complete path, followed by another redirection to the final user-friendly URL.

(From DE ⟶ EN):
www.test.de/detail/1234-haus-am-see/www.test.de/en/detail/1234/www.test.de/en/detail/1234-house-by-the-lake/

(From EN ⟶ DE):
www.test.de/en/detail/1234-house-by-the-lake/www.test.de/en/detail/1234/www.test.de/detail/1234-haus-am-seewww.test.de/detail/1234-haus-am-see/

This behavior needs enhancement to eliminate unnecessary redirects and enhance the SEO-friendliness of the website.

Tasks

  • Analyse the existing code and understand the mechanism governing URL redirection during language switch.
  • Adjust the logic to ensure direct rediretion to the final user-friendly URL without unnecessary intermediate steps.
  • Implement consistent handling of trailing slahes to ensure uniformity in URL structures.
  • Ensure that the changes are backward-compatible and existing URLs are correctly redirected.

The implementation of this change aims to ensure that URL redirection during language switches on my real estate website works seamlessly, contributing to optimal SEO ranking and improved user experience.

redirect-de-zu-en_current JPG (3)
redirect-en-wieder-zu-de_current JPG (3)

@fredericalpers fredericalpers added this to the v4.21 milestone Jan 23, 2024
@dai-eastgate dai-eastgate pinned this issue Jan 30, 2024
@yeneastgate yeneastgate unpinned this issue Jan 30, 2024
@yeneastgate
Copy link
Contributor

yeneastgate commented Jan 30, 2024

@fredericalpers
According to my understanding:
1. Your problem:

2. Your desire:
Transition from German to English URL will redirect directly to the final user-friendly URL?
www.test.de/detail/1234-haus-am-see/ => www.test.de/en/detail/1234-house-by-the-lake/

@fredericalpers
Copy link
Member Author

@yeneastgate yes, we need to eliminate the interim redirection, to always redirect to the final user-friendly URL. The step in between is bad for SEO.

In the video it seems to be working fine :) thank you

@yeneastgate
Copy link
Contributor

@fredericalpers so I need a day to implement and test it. Thanks!

@fredericalpers fredericalpers added small change A change that is small that can be dealt with during ramp-up. and removed investigate labels Jan 30, 2024
@yeneastgate yeneastgate self-assigned this Jan 31, 2024
@yeneastgate
Copy link
Contributor

@fredericalpers I created a PR for this issue. Please take a look at my video demo. Thanks!

cmt_731.mp4

@fredericalpers
Copy link
Member Author

@yeneastgate the video result looks good! is there any chance you can test it with the chrome browser extension Link Redirect Trace and comment the results here aswell? Thank you in advance :)

@yeneastgate
Copy link
Contributor

@fredericalpers Okay, I will record a video and send it back later.

@yeneastgate
Copy link
Contributor

@fredericalpers Here is my video test with the Chrome browser extension.

chrome_extension.mp4

@fredericalpers fredericalpers added QA Issue or Pull request that is in review and removed small change A change that is small that can be dealt with during ramp-up. labels Feb 2, 2024
@fredericalpers fredericalpers added bug Indicates an unexpected problem or unintended behaviour component: property detail page Issue, Pull Request or Discussion related to property detail page component: dashboard Issue, Pull Request or Discussion related to dashboard / settings page labels Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behaviour component: dashboard Issue, Pull Request or Discussion related to dashboard / settings page component: property detail page Issue, Pull Request or Discussion related to property detail page QA Issue or Pull request that is in review
Projects
None yet
2 participants