findissues icon indicating copy to clipboard operation
findissues copied to clipboard

Site links

Open Artal44 opened this issue 1 year ago β€’ 1 comments

PR Template πŸš€

πŸ› Fixed Issue

I tried improving the SEO of the app.

πŸ›  Proposed Changes

In order to improve the SEO, I looked at the site-links improvements. I therefore added a link to a site-map to each page of the website. I wanted the site-map to stay updated even if new improvements or features were added to the app so I added a site-map generator command to config.

πŸ–ΌοΈ Screenshots

Capture d’écran du 2024-01-08 12-42-43

Capture d’écran du 2024-01-08 12-52-40

πŸ“ Notes for Reviewers

I kept the old site-map in a file called "old_site-map" and I found out that the new created site-map was missing some pages. I found the issue. It is because to find the url of the [lang] pages, the generator uses the function getStaticPath of the [lang].js file but this function is only using the langages that are in the priority_langs.js file.

Checklist βœ…

  • [X ] Starred this repo ⭐.
  • [X ] Code adheres to the project's coding standards πŸ’».
  • [X ] No sensitive information is inadvertently exposed πŸ‘€.
  • [X ] Commit history is clear and concise πŸ“ƒ.

Reviewer Tasks πŸ•΅οΈ

Additional Context ℹ️

In order to run the site-map gen, you just have to launch the command : "npm run site-map" this command uses the package "npm install next-sitemap"

Artal44 avatar Jan 08 '24 12:01 Artal44