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

画像をローカルでも表示させるように変更、 2 件のページ内容追記 #61

Merged
merged 18 commits into from
Dec 8, 2023

Conversation

dorimiamn
Copy link
Contributor

@dorimiamn dorimiamn commented Nov 25, 2023

fix #12
fix #21
fix #59
fix #60

  • 画像をマークダウンと同じ階層で管理できるようにした
    • 画像は yarn dev 実行時に、public 配下へとコピーされる。
    • Markdown 中で相対パスで記述したものを、public 配下を参照するように置換する処理
    • 既存のパス指定を相対パスを利用したものに変更
  • tsx ファイル中の img タグについては、src の値を デプロイ時かどうかで切り替えるように設定した
  • markdown の HTML 変換に使うモジュールを一部変更(すでに使っていたものをラップしたモジュールを使う形にした)
  • Windows において、ネットワークアダプターを半角英字に直す手順を追加
  • Intel MacBook 環境にて、 Docker の設定を変更することで期待される動作がなされるよう手順を変更
  • README.md を現場に即した形に修正

すみません、 Markdown の追記と機能の追加は分けて PR を出すべきでした……!
確認しづらくなっておりますが、ご確認のほどどうぞよろしくお願いいたします 🙇

@zenstudysensei
Copy link

💪

@dorimiamn dorimiamn marked this pull request as draft November 28, 2023 07:48
@dorimiamn
Copy link
Contributor Author

Virtualization Framework を無効化するための手順が抜けていたため Draft に

@dorimiamn dorimiamn marked this pull request as ready for review November 28, 2023 08:27
@moesuke
Copy link
Contributor

moesuke commented Dec 8, 2023

めちゃくちゃたすかります!!!

Copy link
Contributor

@moesuke moesuke left a comment

Choose a reason for hiding this comment

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

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

@moesuke moesuke merged commit f88aa84 into main Dec 8, 2023
@moesuke moesuke deleted the fix/59 branch December 8, 2023 09:45
@dorimiamn
Copy link
Contributor Author

ご確認ありがとうございます!!

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