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

the `orgUnitId` of `access-control` aspect minLength should be 0 #3439

Closed t83714 closed 1 year ago

t83714 commented 1 year ago

the orgUnitId of access-control aspect minLength should be 0

The orgUnitId of access-control aspect minLength should be 0 to allow empty string.

https://github.com/magda-io/magda/blob/69cf8e94e9a68af49896bb75c101aaf750407600/magda-registry-aspects/access-control.schema.json#L16

In the ticket:

We have allowed setting records orgUnitId to an empty string as a shortcut to make the record public record (via a permission with orgUnit constraint).

t83714 commented 1 year ago

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