Alberto Islas

Results 199 comments of 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. ![Screenshot 2024-04-30 at 7 38 34 p m](https://github.com/freelawproject/courtlistener/assets/486004/38434766-7af7-430e-8a06-c61ecb877ce7) The `d` and `rd` types lack this key in...