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

Push to PRD #258

Merged
merged 4 commits into from
Feb 3, 2025
Merged

Push to PRD #258

merged 4 commits into from
Feb 3, 2025

Conversation

bitbeckers
Copy link
Contributor

  • Validate attestations using hypercerts SDK
  • Organizing of some imports

Update SDK and contract package for latest deployments and features
Replaces the local token utils with the token utils from the SDK.
They're basically the same but the SDK will be the leading version.
Validate decoded attestation data using the AttestationValidator of the
SDK. This ensure the provided attestation represents a known chain and
contract address. Additionally the tokenID must the the ID of a
claim/base type
@bitbeckers bitbeckers added the enhancement New feature or request label Feb 3, 2025
@bitbeckers bitbeckers self-assigned this Feb 3, 2025
Copy link

github-actions bot commented Feb 3, 2025

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 26.21% (🎯 26%) 1598 / 6095
🟢 Statements 26.21% (🎯 26%) 1598 / 6095
🟢 Functions 69.23% (🎯 69%) 36 / 52
🟢 Branches 73.85% (🎯 73%) 113 / 153
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/parsing/parseAttestationData.ts 90.47% 80% 100% 90.47% 101-106, 120-125
src/parsing/parseAttestedEvent.ts 100% 71.42% 100% 100%
src/parsing/parseTakerBidEvent.ts 100% 95.83% 100% 100%
src/storage/storeTransferSingle.ts 96.96% 50% 100% 96.96% 55-57
Generated in workflow #213 for commit f8a4e17 by the Vitest Coverage Report Action

@bitbeckers bitbeckers merged commit dfa7d3d into main Feb 3, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant