You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now in search-a-licious, you can declare a field as an object, but there is no way you can specify the sub fields of those objects.
It can also lead to problem at import time. For example I got a float field (in nutrients), which was first interpreted as long (maybe because it was 0 in first elements)... in this case ES won't index some of the record because it can't change the field definition.
Proposed solution
Add a fields property to objects fields, that will works as the fields at the config level.
Also add a dynamic bool property to control the behaviour of ES dynamic property on the Object field.
Additional context
This might be needed if we want to handle nutrients correctly as numbers in Open Food Facts (and insure operators works correctly in rqeuests).
This should not be very difficult to implement:
add it in config.py + add validators to ensure we add those properties only for meaningful objects
use this configuration in generate_dsl_field
The text was updated successfully, but these errors were encountered:
Problem
Right now in search-a-licious, you can declare a field as an object, but there is no way you can specify the sub fields of those objects.
It can also lead to problem at import time. For example I got a float field (in nutrients), which was first interpreted as long (maybe because it was 0 in first elements)... in this case ES won't index some of the record because it can't change the field definition.
Proposed solution
Add a fields property to objects fields, that will works as the fields at the config level.
Also add a dynamic bool property to control the behaviour of ES dynamic property on the Object field.
Additional context
This might be needed if we want to handle nutrients correctly as numbers in Open Food Facts (and insure operators works correctly in rqeuests).
This should not be very difficult to implement:
The text was updated successfully, but these errors were encountered: