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.

Sign up for GitHub

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

Jump to bottom

Consistent struct validation #24

Closed
decentralgabe opened this issue Oct 23, 2023 · 0 comments
Closed

Consistent struct validation #24

decentralgabe opened this issue Oct 23, 2023 · 0 comments
Assignees
@finn-block
Labels
did-dht-1.0 enhancement New feature or request

Comments

@decentralgabe
Copy link
Member 

          Possible to add length validation for these 3 fields?

Originally posted by @andresuribe87 in #21 (comment)

Also, helper methods when generating these structs
@decentralgabe decentralgabe added the enhancement New feature or request label Oct 23, 2023
@decentralgabe decentralgabe mentioned this issue Dec 19, 2023
Draft
@finn-block finn-block mentioned this issue Feb 2, 2024
Merged
finn-block added a commit that referenced this issue Feb 6, 2024
@finn-block
reduce the many PkarrRecord structs to one central struct, with valid…
2bc54ff 
…ation and conversions to/from other types

Fixes #24
@decentralgabe decentralgabe moved this to In Progress in SDK Development Feb 6, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in SDK Development Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@finn-block finn-block

Labels
did-dht-1.0 enhancement New feature or request
Projects
SDK Development
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@decentralgabe @finn-block