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

fix: attestation with private-data dataRoot deprecated #113

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

Vib-UX
Copy link
Contributor

@Vib-UX Vib-UX commented Aug 14, 2024

Update Documentation for Schema Encoder to Reflect privateData Usage

Description

This pull request updates the documentation to replace references from dataRoot to privateData in the SchemaEncoder usage. This change aligns with the latest configuration updates in the Ethereum Attestation Service (EAS) SDK.

Changes

  • Modified the documentation to update the SchemaEncoder initialization from 'bytes32 dataRoot' to 'bytes32 privateData'.
  • Adjusted related references in code examples and descriptions to ensure consistency.

Justification

Using privateData as the field name in the SchemaEncoder is now the standard practice according to the updated EAS SDK guidelines. This documentation update ensures our guidance is clear, accurate, and aligned with the latest standards.

Reference

The updated schema configuration using privateData can be viewed here:
View Schema Configuration

Impact

This change is strictly related to documentation and does not affect the actual codebase functionality. It improves clarity for developers using our documentation to implement EAS in their projects.

Testing

Not applicable as the changes are confined to documentation.

Fixes

This PR fixes Issue #112 on the EAS SDK repository.

This update ensures documentation reflects the current practices and supports developers in correctly implementing attestation services using the EAS SDK.

@lbeder
Copy link
Member

lbeder commented Aug 14, 2024

Thanks @Vib-UX!

@lbeder lbeder merged commit 335d38d into ethereum-attestation-service:master Aug 14, 2024
1 check passed
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.

Bug in Private Data Schema Encoder Field Reference in EAS Attestation Script
2 participants