Skip to content

feat: change attestation justification from bytes32 to bytes for unli… - #128

Merged
joelamouche merged 1 commit into
TheSoftwareDevGuild:mainfrom
tusharshah21:fix-33-attestation
Dec 23, 2025
Merged

feat: change attestation justification from bytes32 to bytes for unli…#128
joelamouche merged 1 commit into
TheSoftwareDevGuild:mainfrom
tusharshah21:fix-33-attestation

Conversation

@tusharshah21

Copy link
Copy Markdown
Collaborator

Fixes #33

Change EAS schema justification field from bytes32 (32 byte limit) to bytes (unlimited).
Update encoding/decoding logic in frontend hooks and documentation accordingly.

@joelamouche

Copy link
Copy Markdown
Contributor

This looks good but we can't merge until I have deployed new contracts

@joelamouche

Copy link
Copy Markdown
Contributor

@tusharshah21 I think we already have that in the frontend 🤔 pls resolve conflicts

@tusharshah21

Copy link
Copy Markdown
Collaborator Author

okays on it captain

@joelamouche joelamouche left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Look good! 🚀

@joelamouche

Copy link
Copy Markdown
Contributor

@tusharshah21 pls resolve conflicts

@tusharshah21

Copy link
Copy Markdown
Collaborator Author

updated mate

@joelamouche

Copy link
Copy Markdown
Contributor

"This branch cannot be rebased due to conflicts" 🤔 Are you sure you used rebase?

@tusharshah21

Copy link
Copy Markdown
Collaborator Author

Hoping this covers now!

@joelamouche
joelamouche merged commit 74bf278 into TheSoftwareDevGuild:main Dec 23, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Attestation justification has a limited length

2 participants