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

Asset query service - JSON #3423

Closed
galanisd opened this issue Sep 7, 2023 · 3 comments · Fixed by #3559
Closed

Asset query service - JSON #3423

galanisd opened this issue Sep 7, 2023 · 3 comments · Fixed by #3559
Assignees
Labels
api Feature related to the (REST) api bug Something isn't working
Milestone

Comments

@galanisd
Copy link

galanisd commented Sep 7, 2023

Bug Report

Describe the Bug

Asset query service does not work in all cases when the property_value is a JSON.
See example in #3416

Expected Behavior

See #3416

Observed Behavior

See #3416

Steps to Reproduce

See #3416

Context Information

Add any other context about the problem here.

  • Used version -> EDC v0.2.1

Detailed Description

See #3416

@github-actions
Copy link

github-actions bot commented Sep 7, 2023

Thanks for your contribution 🔥 We will take a look asap 🚀

@ndr-brt ndr-brt added bug Something isn't working api Feature related to the (REST) api labels Sep 7, 2023
@ndr-brt ndr-brt added this to the Milestone 12 milestone Oct 17, 2023
@arnoweiss
Copy link

I've observed the same behavior.

@ndr-brt ndr-brt self-assigned this Oct 18, 2023
@ndr-brt
Copy link
Member

ndr-brt commented Oct 18, 2023

I think the best way to solve this would be to switch the asset properties from the dedicated table we have to a properties and privateProperties json column in the asset table, this way we would enable querying nesting objects.

Currently a maybe good enough workaround would be to use the LIKE operator, as the json is serialized as a string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Feature related to the (REST) api bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants