Skip to content

Conversation

@chvmvd
Copy link
Contributor

@chvmvd chvmvd commented Nov 22, 2025

/src/components/Term/definitions.jsを修正しました。

  • referencePageTitlesプロパティの値を修正して、最新の章構成に合わせました
  • 不要な用語を削除しました。明らかに不要なもののみを削除し、参照されていないだけのものは残しています
  • その他軽微なミスに関しても修正しました

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 22, 2025

Deploying utcode-learn with  Cloudflare Pages  Cloudflare Pages

Latest commit: 06cdf7d
Status: ✅  Deploy successful!
Preview URL: https://dca9a755.utcode-learn.pages.dev
Branch Preview URL: https://update-definitions.utcode-learn.pages.dev

View logs

@chvmvd chvmvd force-pushed the update-definitions branch from 9d8b2cd to 06cdf7d Compare November 22, 2025 00:55
@chvmvd chvmvd marked this pull request as ready for review November 22, 2025 02:31
@@ -1,34 +1,38 @@
export default {
referencePageTitles: {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

現在の章構成と完全に対応するようにしました

referencePage: "/docs/trial-session/html/",
},
fork: {
name: "フォーク",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

フォークは現在扱っておらず、扱う予定もないため削除

referencePage: "/docs/trial-session/get-started/",
},
gitpod: {
name: "Gitpod",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

同様

referencePage: "/docs/trial-session/expressions/",
},
javascriptStringConcatenation: {
name: "文字列の結合",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

使用されていないため削除

name: "ライブラリ",
definition:
"汎用性の高い複数のプログラムを再利用可能な形でひとまとまりにしたもの。基本的に他のプログラムから利用され、単体では実行できない。",
referencePage: "/docs/web-servers/module/",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

3c2a7e7 が原因です

referencePage: "/docs/web-servers/server/",
},
queryString: {
name: "クエリ文字列",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

クエリ文字列は現在扱っておらず、扱う予定もないため削除

referencePage: "/docs/web-servers/database/",
},
thread: {
name: "スレッド",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

同様

Copy link

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 cleans up the term definition system by removing obsolete terms and updating reference page mappings to match the latest documentation structure. The changes consolidate Git/GitHub documentation pages from the trial-session and browser-apps sections into a more organized web-servers section, and remove unused terms like "fork", "Gitpod", "query string", "thread", and "string concatenation".

  • Updated referencePageTitles to reflect renamed, moved, and newly added documentation pages
  • Removed obsolete term definitions ("fork", "Gitpod", "queryString", "thread", "javascriptStringConcatenation")
  • Updated reference pages for existing Git-related terms to point to the new documentation structure

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/components/Term/type-map.js Removed mappings for obsolete terms (fork, Gitpod, query string, thread) that are no longer needed
src/components/Term/definitions.js Updated referencePageTitles to reflect new documentation structure; removed obsolete term definitions and updated reference pages for Git-related terms to point to new locations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@chvmvd chvmvd requested a review from chelproc November 29, 2025 13:20
@chvmvd chvmvd merged commit f678fbc into main Nov 30, 2025
9 checks passed
@chvmvd chvmvd deleted the update-definitions branch November 30, 2025 11:47
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