2.0.0 docs missing migration guide for ref changes
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 52/100
- issue の種類
- ドキュメント
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- typescript
調査の方向性
https://v2.solidjs.com/migration/from-solid-1 の移行ガイドから始め、既存のカバレッジを issue 内の ref コールバックの例と比較してください。ref の所有権とクリーンアップについて、Solid 1.x から 2.x への移行パスを、所有権が変更された理由と、提案されたラッパーアプローチが必要かどうかを含めて文書化してください。
索引モデルが issue の本文から書いたものです。
説明
https://v2.solidjs.com/migration/from-solid-1
<div ref={()=>{
console.log(getOwner()) // 1.x(true) 2.x(false)
}}>
</div>
solid 1.x ref is owned ( https://s.olid.uk/id/qSQe8ZJ6QRyiJTtdfQDNrQ )
solid 2.x ref is non owned ( https://s.olid.uk/id/KyofyVLqQC2upVRrxbCTYw )
In 1.x we could register cleanup in the ref callback, but not in solid 2.x
no ownership is documented in the docs for 2.x . but the migration guide is not addressing it.
it should show the migration path for ref changes.
I assume that we now requires an higher order component wrapper
also what is the reason that ref has no owner?
- 主要言語
- MDX
- スター
- 292
- フォーク
- 361
- 平均マージ
- 7時間 51分
- マージ済み PR（30日）
- 1
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
solidjs/solid-docs のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
solidjs/solid-docs#1635 ·
-
good first issue solid solidstart
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
solidjs/solid-docs#1522 ·
-
pending review
solidjs/solid-docs#1637 · 担当者 1 名 ·
-
pending review
solidjs/solid-docs#1636 · コメント 1 件 · 担当者 1 名 ·
-
help wanted
solidjs/solid-docs#1593 · コメント 4 件 · リアクション 2 件 · 担当者 1 名 ·
solidjs/solid-docs の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
agilepathway/label-checker#640 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
BasedHardware/omi#15662 · コメント 1 件 ·
-
documentation help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
amponce/archive-movie-browser#167 ·
-
user-reported
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
Kong/developer.konghq.com#7316 ·