Skip to content
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

docs: migration from v2 guide #474

Merged
merged 5 commits into from Jun 18, 2022

Conversation

sapphi-red
Copy link
Member

@netlify
Copy link

netlify bot commented Jun 1, 2022

Deploy Preview for vitejs-docs-ja ready!

Name Link
🔨 Latest commit f66a910
🔍 Latest deploy log https://app.netlify.com/sites/vitejs-docs-ja/deploys/62ac954233725200080d4815
😎 Deploy Preview https://deploy-preview-474--vitejs-docs-ja.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 site settings.


- `alias` ([`resolve.alias`](../config/shared-options.md#resolvealias) に置き換え)
Copy link
Member Author

Choose a reason for hiding this comment

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

このあたりのリンクですが、vitejs/vite@5791f5d (#451) に依存していたため、今のところはそのままにしてあります

guide/migration.md Outdated Show resolved Hide resolved
guide/migration.md Outdated Show resolved Hide resolved
guide/migration.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@jay-es jay-es left a comment

Choose a reason for hiding this comment

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

ありがとうございます!:pray:
内容は問題ないのですが、config ページのリンクが dead link でビルドエラーが起きているため #451 が完了してからマージいたします:bow:
(今マージするとしばらくサイト更新が止まってしまうため)


ロジックのほとんどはミドルウェアではなくプラグインのフックを介して行われるべきなので、ミドルウェアの必要性は大幅に減少しています。内部のサーバアプリは、Koa から古き良き [connect](https://github.com/senchalabs/connect) のインスタンスに変わりました
先に Vite の v2 のドキュメントの [v1 からの移行](https://v2.vitejs.dev/guide/migration.html) を確認して、Vite v2 への移行に必要な変更を見てから、このページの変更の適用に移ってください
Copy link
Member Author

Choose a reason for hiding this comment

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

レビューありがとうございます!
失念していたのですが、こちらの https://v2.vitejs.dev/ の日本語版はどうなる予定でしょうか? (現在は英語版へのリンクになっています)

Copy link
Collaborator

Choose a reason for hiding this comment

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

ありがとうございます
他の管理者の方に確認してみます!

@jay-es
Copy link
Collaborator

jay-es commented Jun 17, 2022

@sapphi-red #451 に対応する PR が完了いたしましたので、origin/main を取り込んでいただき、config ページへのリンクを修正お願いします:pray:

@sapphi-red sapphi-red marked this pull request as ready for review June 17, 2022 14:49
@sapphi-red
Copy link
Member Author

guide/migration.mdの以下の行のリンクは元のほうがハッシュ部分が間違っていましたが、このPRに混ぜると変更点が複雑になるため、あえて修正していません (今後、元のほうにPRを出します 👍 )

  • 22
  • 23
  • 28
  • 36
  • 46

また、こちらの「glob インポート」というタイトルなのですが、asという部分が重要なので、このPRとは別で変更するPRを作成しますね

Copy link
Collaborator

@jay-es jay-es left a comment

Choose a reason for hiding this comment

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

ありがとうございます!:pray:

@jay-es jay-es merged commit a8f89e3 into vitejs:main Jun 18, 2022
@sapphi-red sapphi-red deleted the docs/migration-from-v2-guide branch June 19, 2022 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants