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

Require protocol for dwn.records API #825

Open
LiranCohen opened this issue Aug 8, 2024 · 3 comments
Open

Require protocol for dwn.records API #825

LiranCohen opened this issue Aug 8, 2024 · 3 comments

Comments

@LiranCohen
Copy link
Member

In the coming future we are moving completely away from "flat-spaced" DWN records. This means all Records will have to be associated with a protocol.

This will break a lot(putting it lightly) of tests both in the project and externally with our docs and example apps. We should plan on this effort.

Once we require it on the @web5/api layer, it will also be enforced within the dwn-sdk-js library.

@blackgirlbytes
Copy link
Contributor

blackgirlbytes commented Aug 8, 2024

I mentioned this move to my teammates on the DevRel team, and we're concerned that while this can make the DevEx easier for advanced users of Web5 like sondreb, we will lose a lot of people if they're required to understand protocols to get started. I'd love to have a more in-depth conversation on this.

@angiejones
Copy link
Contributor

@LiranCohen mind providing the rationale for this change?

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

No branches or pull requests

3 participants