search

SEMICeu / LDES-DCAT-AP-feeds

This repository will host the ongoing work regarding LDES and DCAT-AP feeds
https://semiceu.github.io/LDES-DCAT-AP-feeds/
3 stars 0 forks source link

Remove or Delete #5

Closed pietercolpaert closed 3 months ago

pietercolpaert commented 4 months ago

There are slightly different semantics for remove and delete. Review the different sema tics and decide which one to use.

I think as:Delete might be the right one after all, as we delete the entity from existence, and not just from this catalog?

https://www.w3.org/TR/activitystreams-vocabulary/

pietercolpaert commented 3 months ago

@smessie Looked into this:

I think it depends on the meaning you intend to transfer. I feel like the opposite of Create is Delete, and the opposite of Add is Remove, making Delete the preferred one here. If you say that you delete the entity from existence, and not just from the catalog, I think that confirms the choice for Delete. However, no matter what is chosen in the end, Remove or Delete, I feel like the same vocabulary should be used in the spec, i.e. we should speak of "Delete" instead of "Remove" in the spec if we go with Delete. Just to avoid confusion.

I follow this and we must change the spec to as:Delete.

pietercolpaert commented 3 months ago

Fixed in https://github.com/SEMICeu/LDES-DCAT-AP-feeds/commit/d90bbec30d4bd3cbbb6c253b59462685cc5da510