Skip to content

✨(backends) add support for queries with multiple authorities - #638

Open
piptouque wants to merge 1 commit into
openfun:mainfrom
piptouque:feat_statement_get_multiple_authorities
Open

piptouque wants to merge 1 commit into
openfun:mainfrom
piptouque:feat_statement_get_multiple_authorities

Conversation

@piptouque

Copy link
Copy Markdown
Contributor

Purpose

Specifically, added the option for RalphStatementQuery.authority to be a list.
Added support for that in each backend: ElasticSearch, MongoDB, Clickhouse and Filesystem.
Note that this is for internal Ralph queries to the backend, it does not change Ralph's API.
In practice, it allows us to query statements coming from multiple authorities at once (OR's).
It's a requirement for another feature that I worked on. Feel free to close if it's not desired.

Proposal

  • Change RalphStatementQuery.authority type to a list
  • Update relevent _add_agent_filters method in each backend
  • add relevent tests cases in unit tests

@piptouque
piptouque force-pushed the feat_statement_get_multiple_authorities branch 6 times, most recently from a61e18f to 0559bda Compare September 7, 2026 14:23
Specifically, added the option for
`RalphStatementQuery.authority` to be a list.
Added support for that in each backend.
@piptouque
piptouque force-pushed the feat_statement_get_multiple_authorities branch from 0559bda to 829cec2 Compare September 17, 2026 09:53

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant