magda-io / magda

A federated, open-source data catalog for all your big data and small data
https://magda.io
Apache License 2.0
505 stars 92 forks source link

Dataset Metadata Editor: Better records updating logic #3421

Closed t83714 closed 1 year ago

t83714 commented 1 year ago

We have made some improvements to the records updating logic in ticket:

The previous includes some improvement to the records updating logic at frontend.

However, pure frontend effort won't be able to address all edging cases.

e.g. Because the current PUT /api/v0/registry/records/{recordId} only supports the replace behaviour, it requires API invoker always to include all record data (including access-control-aspect).

This might not always be feasible when you only want to update certain fields of the aspect without touching other fields.

Once this ticket:

Changes should be properly test to make sure issue #3412 will not break again

t83714 commented 1 year ago

closed via PR: https://github.com/magda-io/magda/pull/3423