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

Epic 1705: Make the daily rollup a regular report search #1733

Closed
maradragan opened this issue May 14, 2019 · 1 comment
Closed

Epic 1705: Make the daily rollup a regular report search #1733

maradragan opened this issue May 14, 2019 · 1 comment
Assignees
Labels

Comments

@maradragan
Copy link
Contributor

maradragan commented May 14, 2019

The daily rollup should use a regular report search query. This also means that it can just use the report search filters.
Part of #1705

@maradragan maradragan changed the title Move date filters daily rollup to the search bar Make the daily rollup a regular report search May 14, 2019
@maradragan maradragan changed the title Make the daily rollup a regular report search Epic 1705: Make the daily rollup a regular report search Jul 8, 2019
@gjvoosten
Copy link
Collaborator

The rollup query currently returns something like:

{
	"data": {
		"rollupGraph": [
			{
				"org": {
					"uuid": "8824e2df-e3b4-46a7-ac31-d98bed31ae5c",
					"shortName": "EF 1"
				},
				"published": 2,
				"cancelled": 0
			},
			{
				"org": {
					"uuid": "b24f99af-2827-4c17-a39d-1f150552af72",
					"shortName": "EF 2"
				},
				"published": 10,
				"cancelled": 1
			},
			…etc…
		]
	}
}

i.e. aggregated numbers (count). Having the search return aggregated numbers will be difficult.

We might use reports search to return the list of (published or cancelled) reports for each top-level organisation (including children), and that could even use batching, but that would mean the client-side would have to do the aggregation (count). In addition to that, getting this all at once for all top-level organisations is not so simple, either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants