search

TBD54566975 / web5-js

Monorepo for the Web5 JS TypeScript implementation
https://tbd54566975.github.io/web5-js/
Apache License 2.0
133 stars 54 forks source link

Require `protocol` for `dwn.records` API #825

Open LiranCohen opened 2 months ago

LiranCohen commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

@LiranCohen mind providing the rationale for this change?

blackgirlbytes commented 2 months ago