Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

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

Đang mở
  • Có gì không ổn?
#47 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
54/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
backend

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
Python
Star
2
Fork
4
Merge trung bình
6 giờ 54 phút
Pull request đã merge (30 ngày)
7

Chuẩn bị môi trường

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của QMeshPy/distributed-quantum

Tất cả issue của QMeshPy/distributed-quantum

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.