-
Notifications
You must be signed in to change notification settings - Fork 56
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
Comments
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. |
@LiranCohen mind providing the rationale for this change? |
|
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 thedwn-sdk-js
library.The text was updated successfully, but these errors were encountered: