Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

chore(vault-pinning): remove the legacy nft-storage alias

Offen
  • Stimmt etwas nicht?
#47 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
54/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
typescript
Bereich
backend

Rechercherichtung

Start with types.ts, services/nft-storage.ts, services/index.ts, and hooks/use-pin.ts to trace how pinning services are selected and how audit records are read and unpinned. Then inspect the existing tests for those paths and add coverage for records with service: "nft.storage". Done means the alias is removed from the type and services, the hook uses DEFAULT_SERVICE, and legacy records still display and unpin correctly.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement good first issue helia storage tier-1

Summary

nft.storage is still a valid PinningService value, but the adapter behind it is really Pinata.

Estimated size: S

Current state in this repo

  • types.ts: PinningService = "nft.storage" | "pinata" | "lighthouse".
  • services/nft-storage.ts reads NEXT_PUBLIC_PINATA_JWT, and its header comment says NFT.Storage is a Filecoin archival service.
  • hooks/use-pin.ts defaults service to "nft.storage" while DEFAULT_SERVICE in services/index.ts is "pinata".

Proposal

  • Remove the alias from the type, delete the adapter, and make the hook default to DEFAULT_SERVICE.
  • Decide and document how existing audit records with service: "nft.storage" are read (map to pinata at read time, or migrate).

Acceptance criteria

  • No nft.storage value remains in the type or services.
  • Old audit records still display and unpin correctly.
  • Tests cover the legacy record case.

Dependencies

  • None; can be picked up independently.

Testing plan

  • Unit test for the read-time mapping.

Part of #43. Ground rules for the content layer: no secrets in the repo, content-layer failures must never block or discard quantum execution, and tests and docs ship with every PR.

Vorherrschende Sprache
Python
Sterne
2
Forks
4
Ø Merge
5 Std. 19 Min.
Gemergte PRs (30 T.)
11

Entwicklungsumgebung

  • Enthält ein Dockerfile oder eine Docker-Compose-Datei
  • Keine Pull-Request-Vorlage
  • Kein Beitragsleitfaden

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus QMeshPy/distributed-quantum

Alle Issues in QMeshPy/distributed-quantum

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

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