Skip to content

Conversation

Kenzo-Wada
Copy link
Contributor

Description (required)

japanese translation for //recipes/build-forms-api/

Copy link

netlify bot commented Sep 25, 2025

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 977e4ca
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/68eaa6f9b8bf1c0008b14a36
😎 Deploy Preview https://deploy-preview-12421--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Sep 25, 2025
@astrobot-houston
Copy link
Contributor

astrobot-houston commented Sep 25, 2025

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
ja/recipes/build-forms-api.mdx Localization added, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@Copilot Copilot AI review requested due to automatic review settings October 8, 2025 14:42
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a Japanese translation for the "build-forms-api" recipe documentation, translating content from English to Japanese while maintaining the same technical structure and code examples.

  • Complete Japanese translation of the build-forms-api recipe documentation
  • Preserves all original code examples and technical content
  • Maintains proper markdown formatting and component structure

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines 27 to 45
```ts title="src/pages/api/feedback.ts" "request.formData()" "post"
export const prerender = false; // Not needed in 'server' mode
import type { APIRoute } from "astro";

export const POST: APIRoute = async ({ request }) => {
const data = await request.formData();
const name = data.get("name");
const email = data.get("email");
const message = data.get("message");
// Validate the data - you'll probably want to do more than this
if (!name || !email || !message) {
return new Response(
JSON.stringify({
message: "Missing required fields",
}),
{ status: 400 }
);
}
// Do something with the data, then return a success response
Copy link
Member

Choose a reason for hiding this comment

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

There are a few comments missing translation here, could you take a look and translate them? 🙌

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed: 99d6a54

この例ではメッセージを含むJSONオブジェクトをクライアントへ返します。

```ts title="src/pages/api/feedback.ts" "request.formData()" "post"
export const prerender = false; // Not needed in 'server' mode
Copy link
Member

Choose a reason for hiding this comment

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

Only one comment missing @Kenzo-Wada, then we are good to go!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants