search

xregistry / spec

xRegistry related specifications
https://xRegistry.io
Apache License 2.0
37 stars 6 forks source link

Update model via `PUT /` - support? #97

Closed duglin closed 5 months ago

duglin commented 6 months ago

Today people can update the model via PUT /model and PUT / + data+model in the body

Seems odd that someone can't define the registry and upload the data in one import operation when they can export it in one.

Proposal: allow it

We could choose to not support it for v1 but we need to decide whether to consider it for v1 or not

duglin commented 6 months ago

agreed on the 4/3 call to allow it

duglin commented 6 months ago

Follow-on question, does PUT require the use of ?model similar to how we require ?inline so people don't change it by accident by doing a GET + PUT w/o realizing they're updating the model? I think we should

duglin commented 6 months ago

Added requirement for ?model