Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to some list fields to be of type array or null #192

Open
1 task done
jh-RLI opened this issue Oct 29, 2024 · 1 comment
Open
1 task done

Add option to some list fields to be of type array or null #192

jh-RLI opened this issue Oct 29, 2024 · 1 comment
Assignees
Labels
part: backend 🧱 Backend component priority: low 🦥 Low priority status: blocked 🛑 Blocked or impeded progress type: enhancement ⚙️ Improvement of an existing feature

Comments

@jh-RLI
Copy link
Contributor

jh-RLI commented Oct 29, 2024

Description of the issue

Some fields like "isAbout" have to be of type array. If there is no element, then this field must evaluate to an empty list ([ ]) element. This is fine but creates some specific issues that are raised due to how things are currently implemented in the oeplatform.

If one create metadata for a table resource, then the oeplatform currently sets empty fields to "null". This is valid with the current release of OMI, but since OMI is reworked and now fully depends on the oemetadata´s json schema this structure is now invalid.

As far as im aware json schema and frictionless are okay with, still this could raise confusion on why the fields can be empty and if the information is even relevant in this case.

Ideas of solution

TBD: provide list of nullable fields
Add nullable option to json schema

Workflow checklist

@jh-RLI jh-RLI added the type: enhancement ⚙️ Improvement of an existing feature label Oct 29, 2024
@jh-RLI jh-RLI self-assigned this Oct 29, 2024
@Ludee
Copy link
Member

Ludee commented Nov 13, 2024

Is this issue solved with the new version?

@Ludee Ludee added priority: low 🦥 Low priority part: backend 🧱 Backend component status: blocked 🛑 Blocked or impeded progress labels Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part: backend 🧱 Backend component priority: low 🦥 Low priority status: blocked 🛑 Blocked or impeded progress type: enhancement ⚙️ Improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

2 participants