-
Notifications
You must be signed in to change notification settings - Fork 18
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
Gatsby -> Astro 移行 #1309
Merged
Gatsby -> Astro 移行 #1309
Conversation
This file contains 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
TableWrapper、CaptureImageWithDescを移植
textlintを追加
画像のフォーマットと圧縮率を変更
フィードバック分の修正
MonthPickerのサムネイルを追加
arrow2nd
requested review from
versionfive,
tosiiu and
a team
as code owners
November 15, 2024 01:32
uknmr
approved these changes
Nov 15, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GoGo!
oremega
approved these changes
Nov 15, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No Look LGTM
tosiiu
approved these changes
Nov 15, 2024
wentzzz
approved these changes
Nov 15, 2024
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.
やったこと
変更箇所
ディレクトリ
content/articles
→src/content/articles
に/static
→/public
にコンポーネントのimport周り
@Components/
から@/components
に@/components/article/*
に集約ツールチェーン周り
yarn
→pnpm
styled-componnets
→ 一部を残してCSS Modules
+Sass
ts-node
→ tsxtype: "module"
なスクリプトを実行するためhusky
で push 前にastro check
が通るか確認するようにデータの取得
SmartHR UIとChromaticのデータ (gatsby-source-ui-versions)
./scripts/fetch-ui-data.ts
をビルド前に実行して取得するように./src/cache/
にJSONで保存されますpnpm dev
pnpm build
すると必ず実行されるので、直接実行する必要はありませんStorybookのキャプチャ (gatsby-source-component-captures)
./scripts/component-thumbnails.ts
./src/lib/fetchComponentCaptures.ts
で行なうように<ComponentCaptures>
内のみなので、キャッシュ等の仕組みは実装していません<ComponentStory>
で表示するStoryのコードページのサイドバー (メニュー / 目次) 周り
左側のメニュー
order
が同一の場合順不同に右側の目次
目次の生成をビルド時に行うように
Algoliaのインデックスの更新
./scripts/update-algoliasearch.ts
を実行して行なうようにMDXの記法
開く
太字
文章中に ()や「」など記号を含む太字 を挿入する場合は、左右にスペースが必要に
エスケープが必要な文字
{} を表示するには、
\{\}
のようにバックスラッシュでのエスケープが必要にコメント
HTML形式ではなくJSX形式に
textlintのコメントも同様です
リンク
リンク先が外部の場合は自動で別タブで開くように (aタグが不要に)
コードブロック
codeBlock
の指定が不要にインラインコードは今まで通り、
<code>
or `` です記事内で使用するコンポーネントの仕様変更
直接コンポーネントを定義している箇所
開く
ReactのAPIを使っていたり、PropsでReactコンポーネントを受け取るコンポーネントは別ファイルに切り出して使うように
❌️
⭕️
StaticImage
astro:assets
の Image コンポーネントを使うようにDoAndDont
img
に要素を渡すのではなく、 要素にslot="img"
を付けるようにlabel
に要素を渡すのではなく、要素にslot="label"
を付けるようにPageIndex
<Description>
が<div>
に (任意のタグが利用可能)name=
からslot=
で指定するように削除箇所
UIバージョンの切り替え機能
ツールチェーン周り
stylelint
改善・修正箇所
/products/usability/accessibility/
を開いた際にヘッダーのアクセシビリティ、プロダクト両方が選択状態になる/products/components/time-picker%20/
->/products/components/time-picker/
に