Express の章の再構成 #787
Merged
Express の章の再構成 #787
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Deploying utcode-learn with
|
| Latest commit: |
2e78852
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://5525f57f.utcode-learn.pages.dev |
| Branch Preview URL: | https://restructure-express.utcode-learn.pages.dev |
chelproc
force-pushed
the
restructure-express
branch
from
655c087 to
ad49567
Compare
aster-void
reviewed
Jun 1, 2024
|
|
||
| app.get("/", (request, response) => { | ||
| response.send("Hello World"); | ||
| response.send('Hello World <a href="./lang/ja">日本語</a>'); |
Contributor
There was a problem hiding this comment.
他の場所では Hello World! となっているので、統一してください。
aster-void
reviewed
Jun 1, 2024
| - `href` 属性が相対パスの形式で指定されていれば、現在のパスを基準に [Linux における相対パス](/docs/web-servers/linux-commands/#パス)と同様の方法で URL を更新します。この例では、更新前のパスが `/` で、`href` 属性には `./lang/ja` が指定されているため、更新後のパスは `/lang/ja` になり、URL は `http://localhost:3000/lang/ja` に変化します。 | ||
| 8. 2 から 5 までの手順が、`/lang/ja` というパスに対して行われます。 | ||
|
|
||
| ### 確認問題 |
Contributor
Author
|
@aster-void 細かいところは後で対応するので構成が問題ないかだけ先に確認してほしいです！ |
chvmvd
reviewed
Jun 2, 2024
| [`Request`](https://expressjs.com/ja/api.html#req) オブジェクトにどのような情報が含まれているか覗いてみましょう。 | ||
|
|
||
| ::: | ||
| ## HTML に指定された外部リソースの取得 |
Contributor
There was a problem hiding this comment.
ここの章の終わりに最終的に伝えたいことを明示してもいいかもしれません。
「わかったけど、だから？」となる人が出そうな気がします。
Contributor
|
全部見て、議題に上がりそうなところおよび、気になったところをコメントしました。他は良いかと思います。 |
chelproc
force-pushed
the
restructure-express
branch
from
1da8e92 to
9be73c6
Compare
chelproc
force-pushed
the
restructure-express
branch
from
9be73c6 to
af883ea
Compare
nakaterm
approved these changes
Jun 2, 2024
chvmvd
reviewed
Jun 2, 2024
docs/3-web-servers/05-server/_samples/app-use/static/index.html
Outdated
Show resolved
Hide resolved
docs/3-web-servers/05-server/_samples/app-use/static/index.html
Outdated
Show resolved
Hide resolved
docs/3-web-servers/05-server/_samples/app-use/static/sub/index.html
Outdated
Show resolved
Hide resolved
docs/3-web-servers/05-server/_samples/app-use/static/sub/script.js
Outdated
Show resolved
Hide resolved
chelproc
force-pushed
the
restructure-express
branch
from
eb3c323 to
2e78852
Compare
Contributor
|
リリースしちゃうので、マージします。 |
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Express の章を大幅に改定し、HTTP に対する理解を深められる内容にした。