Skip to content

Fix: missing localizations in middleware populate#98

Merged
alex-strapi merged 10 commits into
mainfrom
fix/90-locales
Feb 17, 2026
Merged

Fix: missing localizations in middleware populate#98
alex-strapi merged 10 commits into
mainfrom
fix/90-locales

Conversation

@fonodi
Copy link
Copy Markdown
Collaborator

@fonodi fonodi commented Dec 11, 2025

What does it do?

Fixes the missing localizations key in middleware populate. Also added some cleanup.

Why is it needed?

Because of this the locale switcher was not showing up in the main header section.

How to test it?

When going on the page, the locale switcher should show the available locales from Strapi.

Some additional things to check:

  • Strapi project uuid is "LAUNCHPAD". strapi/packages.json.
  • Strapi version is the latest possible.
  • If the Strapi version has been changed, make sure that the strapi/scripts/prefillLoginFields.js works.
  • If you updated content, make sure to create a new export in the strapi/data folder and update the strapi/packages.json seed command if necessary.

Related issue(s)/PR(s)

Let us know if this is related to any issue/pull request.

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 11, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
launch-pad Ready Ready Preview, Comment Feb 17, 2026 10:57am
launch-pad-mzzo Ready Ready Preview, Comment Feb 17, 2026 10:57am

Request Review

@fonodi
Copy link
Copy Markdown
Collaborator Author

fonodi commented Dec 11, 2025

@PaulBratslavsky while doing #90 , I figured out that the localizations was in the middleware populates. Nothing threw an error and it defaulted to an empty array. That's why the the header had no locale switcher.

I also updated Strapi to the latest version as said in the todos and ran some shai-hulud 2.0 scans against the package.json and the repo, which threw no errors.

I saw some potential to simplifie and cleanup some code as well, which I also included here. Please let me know what you think about it 👍

@fonodi fonodi changed the title Fix: missing Fix: missing localizations in middleware populate Dec 11, 2025
@fonodi fonodi requested a review from alex-strapi December 15, 2025 13:31
Copy link
Copy Markdown
Collaborator

@alex-strapi alex-strapi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for removing patch-package too!

Copy link
Copy Markdown
Collaborator

@PaulBratslavsky PaulBratslavsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fonodi looks great, thank you, you can merge it.

Copy link
Copy Markdown
Collaborator

@PaulBratslavsky PaulBratslavsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙂

@alex-strapi alex-strapi merged commit acb173f into main Feb 17, 2026
3 checks passed
@alex-strapi alex-strapi deleted the fix/90-locales branch February 19, 2026 23:48
jadenblack pushed a commit to indigo-services/strapi-launchpad that referenced this pull request May 26, 2026
Fix: missing `localizations` in middleware populate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants