Alberto Islas
Alberto Islas
> Ideally, only the opinions that match should show in the results. Yeah, this is how it currently works. > If a cluster matches (but the opinion doesn't), then showing...
> Displaying all the subopinions in that case is fine too! Great, I've set the limit for sub-opinions to 20. Hoping this limit is enough to display all the possible...
Sure, I've resolved the conflicts and added the `meta` key to the Opinions serializers as well. So this is now ready for review!
thanks! @ERosendo I've applied your suggestions. Also, I've converted list fields in `Opinions` and `RECAP` to `NoneToListField` and added tests to handle the bug related to ES DSL partial updates.
> Yeah, that makes sense. Do we have the docket_entry_id in the recap_docket object in ES also? Correct, I'll add the `docket_id` to the response. And yeah, the `docket_entry_id` is...
> That sounds right, but to be certain I understand correctly, are there fields that are shown on both the RD and the R search type that are not highlighted...
Great, I've added the **`docket_id`** field as part of the response for the "rd" search type. I've also changed the **`count`** key to display the exact number of results if...
> One other thought: Is it hard to add a second count to the r results? One for the docket count and one for the recap document count, like we...
> I think it's OK to add it to all responses, particularly if we do the cardinality thing that should make it pretty performant? Yeah! I'll add the secondary count...
The `document_count` provided by a cardinality query is added to the `r` type.  The `d` and `rd` types lack this key in...