solidjs / solidjs/solid-docs

2.0.0 docs missing migration guide for ref changes

Offen
#1,630 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
MDX
Sterne
292
Forks
361
Ø Merge
7 Std. 51 Min.
Gemergte PRs (30 T.)
1

Beschreibung

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?

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit dem Migrationsleitfaden unter https://v2.solidjs.com/migration/from-solid-1 und vergleiche dessen bestehende Abdeckung mit den ref-Callback-Beispielen im Issue. Dokumentiere den Migrationspfad von Solid 1.x zu 2.x für ref-Ownership und Cleanup, einschließlich des Grundes für die Änderung der Ownership und der Frage, ob der vorgeschlagene Wrapper-Ansatz erforderlich ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
documentation
Issue-Typ
Dokumentation
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
52/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.