Skip to content

Conversation

@fonodi
Copy link
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

vercel bot commented Dec 11, 2025

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

Project Deployment Review Updated (UTC)
launch-pad Ready Ready Preview, Comment Dec 15, 2025 2:35pm
launch-pad-mzzo Ready Ready Preview, Comment Dec 15, 2025 2:35pm

@fonodi
Copy link
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
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!

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.

2 participants