Normalize i18n notice translation

[ShubhamOulkar]

closes #1768

related pr #1779

Right now it shows an empty notice box, but the notice will populate once translation is updated by Crowdin.

[netlify]

✅ Deploy Preview for expressjscom-preview ready!

Name	Link
🔨 Latest commit	cd1fba0
🔍 Latest deploy log	https://app.netlify.com/projects/expressjscom-preview/deploys/6837438687ad0c0008a0ec53
😎 Deploy Preview	https://deploy-preview-1930--expressjscom-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[bjohansebas]

I think we need to do this step manually for now, since it would be unusual for the alert to appear without text or we could probably implement some logic to check if the string exists, and if not, fall back to the English one

[ShubhamOulkar]

fall back to the English one

added in 1b42b97

[bjohansebas]

I guess we won’t be able to use some Liquid/Jekyll methods, maybe we’ll have to put the text directly and upload the file to Crowdin.

[ShubhamOulkar]

maybe we’ll have to put the text directly and upload the file to Crowdin.

  <a href="{{ page.url | replace: '/en/', '/en/' }}">documentation in English</a>.

That was the doubt I forgot to include in the description. Maybe doing it manually in Crowdin is the only option.

[bjohansebas]

That was the doubt I forgot to include in the description. Maybe doing it manually in Crowdin is the only option.

We can’t, because the link is dynamic.

[bjohansebas]

Although we can probably remove the banner, right? @expressjs/docs-wg WDYT?

[ShubhamOulkar]

The issue is fixed in cd1fba0.

[bjohansebas]

LGTM, let's leave this open a bit longer.

[bjohansebas]

thanks @ShubhamOulkar!