Update Ceramic metadata field to use $comment field

ceramicnetwork / CIPs

The Ceramic Improvement Proposal repository

https://cips.ceramic.network/

MIT License

82 stars 22 forks source link

Update Ceramic metadata field to use $comment field #104

Closed PaulLeCam closed 3 years ago

PaulLeCam commented 3 years ago

Uses $comment instead of $ceramic field in JSON schemas
Replaces mentions of document/DocID by stream/StreamID
Uses tags metadata for deterministic slice access for append-collection

stbrody commented 3 years ago

I've laid out my thoughts as best as I can, but there are still 2 open comment threads which need some final decisions made. I just made a last comment on each thread summarizing my current thinking. Now I think we need @oed to weigh in.

PaulLeCam commented 3 years ago

@oed @stbrody do you think we could merge this as-is for now please? We can always update the specs based on the implementation changes.

oed commented 3 years ago

@PaulLeCam made a proposal for how to use our existing url encoding for streamids

oed commented 3 years ago

What's the status of this effort @PaulLeCam? I think an update is needed for https://github.com/ceramicnetwork/CIP/issues/87

PaulLeCam commented 3 years ago

IIRC it should be good to merge, the main change that could be required is replacing the use of tags for AppendCollection but anyways the AppendCollection CIP will likely need other updates as we decide on an implementation.