Skip to content
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

feat/metadata-field-hip-766 #64

Merged
merged 17 commits into from
Jun 4, 2024

Conversation

michalrozekariane
Copy link
Contributor

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@michalrozekariane michalrozekariane self-assigned this May 9, 2024
Copy link

github-actions bot commented May 9, 2024

Jest Node 20 Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
1 files    0
1 errors

Results for commit d4dd707.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 9, 2024

Jest Node 18 Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
1 files    0
1 errors

Results for commit d4dd707.

♻️ This comment has been updated with latest results.

@michalrozekariane michalrozekariane changed the base branch from main to feat/metadata-key May 10, 2024 13:33
README.md Outdated
@@ -30,6 +30,7 @@ This package includes all sorts of tooling for the Hedera NFT ecosystem, includi
- **PinataService:** Utilizes Pinata Cloud to pin files to IPFS, providing an `ipfs://` URL upon successful upload. Includes metadata and options customization.
- **NftStorageService:** Integrates with the NFT.storage API to upload files directly to IPFS and returns an `ipfs://` URL. It supports dynamic API key usage based on a provided list.
- **MockStorageService:** A mock storage service for testing purposes, returning predefined URLs.
15. **Collection Metadata Validation** Provides a possibility to validate the collection metadata of a collection against the HIP766 metadata schema. This method can handle both direct HTTP(S) URLs or IPFS CIDs as input. If an IPFS CID is provided without a full URL, the method will attempt to use an IPFS gateway to retrieve the metadata. The IPFS gateway parameter is optional; if not specified and required (i.e., when a CID is provided without a full URL), the method will throw an error indicating that the IPFS gateway is required. This ensures that the metadata conforms to the standards required for NFT collections on the Hedera network. The validator provides a thorough check, highlighting any errors or missing fields in the collection metadata structure.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change HIP766 to HIP-766 and add a hyperlink to https://hips.hedera.com/hip/hip-766

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

README.md Outdated
@@ -1812,6 +1814,39 @@ type UploadResult = {

---

## Collection Metadata Validation

The `validateCollectionMetadata` validates the metadata of a collection against the [HIP766 metadata schema](https://hips.hedera.com/hip/hip-766) using a specified IPFS gateway to retrieve the metadata. This method ensures that the metadata conforms to the standards required for NFT collections on the Hedera network, offering a comprehensive check that highlights any errors or missing fields in the metadata structure.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change HIP766 to HIP-766

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

README.md Outdated

This method returns an object that contains:

- `isValid`: A boolean flag indicating whether the metadata conforms to the HIP766 schema.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change HIP766 to HIP-766

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Signed-off-by: michalrozekariane <[email protected]>
@kacper-koza-arianelabs kacper-koza-arianelabs marked this pull request as ready for review June 4, 2024 14:08
Signed-off-by: kacper-koza-arianelabs <[email protected]>
@hashgraph hashgraph deleted a comment from KacperKoza343 Jun 4, 2024
Signed-off-by: kacper-koza-arianelabs <[email protected]>
@kacper-koza-arianelabs kacper-koza-arianelabs force-pushed the feat/metadata-field-hip-766 branch from 398f480 to d4dd707 Compare June 4, 2024 14:34
@kacper-koza-arianelabs kacper-koza-arianelabs merged commit f9ee518 into feat/metadata-key Jun 4, 2024
2 checks passed
@kacper-koza-arianelabs kacper-koza-arianelabs deleted the feat/metadata-field-hip-766 branch June 4, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants