-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
chore(wsl): add notice #7758
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
base: main
Are you sure you want to change the base?
chore(wsl): add notice #7758
Changes from 3 commits
20150a0
a314703
686ee4b
6123fef
b2cf976
3ec4be3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
'use client'; | ||
|
||
import { useTranslations } from 'next-intl'; | ||
import type { FC } from 'react'; | ||
|
||
const WSLMessage: FC = () => { | ||
const t = useTranslations(); | ||
return ( | ||
<> | ||
<div className="mt-4 rounded-lg border border-blue-200 bg-blue-50 p-4 dark:border-blue-800 dark:bg-blue-900/20"> | ||
<h3 className="mb-2 text-lg font-semibold text-blue-800 dark:text-blue-200"> | ||
{t('layouts.download.wsl.title')} | ||
</h3> | ||
<p className="text-blue-700 dark:text-blue-300"> | ||
{t('layouts.download.wsl.description')} | ||
</p> | ||
<a | ||
href="https://docs.microsoft.com/en-us/windows/wsl/install" | ||
target="_blank" | ||
rel="noopener noreferrer" | ||
className="mt-2 inline-block text-blue-600 hover:underline dark:text-blue-400" | ||
> | ||
{t('layouts.download.wsl.learnMore')} | ||
</a> | ||
</div> | ||
</> | ||
); | ||
}; | ||
export default WSLMessage; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
'use client'; | ||
|
||
import { useContext } from 'react'; | ||
|
||
import { ReleaseContext } from '#site/providers/releaseProvider'; | ||
|
||
import WSLMessage from './WSLMessage'; | ||
|
||
const WindowsWSLMessage = () => { | ||
const { os } = useContext(ReleaseContext); | ||
// consitioms to chekc ifo the operating system is selcted as Linux or not | ||
if (os !== 'WIN') return null; | ||
return <WSLMessage />; | ||
}; | ||
|
||
export default WindowsWSLMessage; |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,8 @@ layout: download | |
title: Download Node.js® | ||
--- | ||
|
||
<WindowsWSLMessage /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This can be part of the Layout, right? And can't AlertBox be used instead of a new component? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
It can say something like: "These instructions are meant to be ran in a PowerShell terminal. If you are using [Windows Subsystem for Linux](link to ms docs), please refer to the Linux instructions" There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry but I couldn't find the "Community Installation" method / option at nodejs website! can you please guide me through url or something so that I can get a rough idea about AlertBox There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @Vishal-K-988 If you select 'Windows' and 'Chocolatey' you should see the AlertBox component (it's the blue info box). |
||
|
||
<section> | ||
|
||
Get Node.js® <Release.VersionDropdown /> for <Release.OperatingSystemDropdown /> using <Release.InstallationMethodDropdown /> with <Release.PackageManagerDropdown /> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,8 @@ layout: download | |
title: Descarga Node.js® | ||
--- | ||
|
||
<WindowsWSLMessage /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also these changes can be reverted, only the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Crowdin will handle for rest of the supported languages, Right ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done & Committed ! |
||
|
||
<section> | ||
Obtiene Node.js® <Release.VersionDropdown /> para <Release.OperatingSystemDropdown /> usando <Release.InstallationMethodDropdown /> con <Release.PackageManagerDropdown /> | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -282,6 +282,11 @@ | |
"devbox": "Devbox는 격리되고 재현 가능한 개발 환경을 제공합니다.", | ||
"docker": "Docker는 컨테이너화 플랫폼입니다. " | ||
} | ||
}, | ||
"wsl": { | ||
"title": "Windows를 사용 중이신가요? WSL을 사용해 보세요 🦖", | ||
"description": "Windows Subsystem for Linux (WSL)을 사용하면 가상 머신 없이 Windows에서 직접 Linux 환경을 실행할 수 있습니다. 이는 Node.js 애플리케이션 개발 경험을 향상시킬 수 있습니다.", | ||
"learnMore": "WSL에 대해 자세히 알아보기" | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you only change the English files? Crowdin will update the other languages eventually There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Removing all the changes made to all the other languages! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I recently saw the vercel deployment and when changed the language to Japanese, still shows the message in english There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That's okay. Crowdin handles translations |
||
} | ||
} | ||
|
Uh oh!
There was an error while loading. Please reload this page.