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

Data source for the Analytics App #2195

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Data source for the Analytics App #2195

wants to merge 13 commits into from

Conversation

Rello
Copy link

@Rello Rello commented Jun 1, 2024

This pull adds a data source to report on forms with Analyitcs

  • real time reporting
  • scheduled data loads (for trends)
  • charts and/or tables
  • multi page dashboards (coming with Analytics 5)
  • PDF/print dashboards (coming with Analytics 5)
  • one-click dashboard creation for all questions of a form

Data source selection

Bildschirmfoto 2024-06-01 um 17 43 25


Single report

Bildschirmfoto 2024-06-01 um 17 46 31


Dashboard

Bildschirmfoto 2024-06-01 um 17 48 57

Copy link

codecov bot commented Jun 1, 2024

Codecov Report

Attention: Patch coverage is 0% with 53 lines in your changes missing coverage. Please review.

Project coverage is 45.17%. Comparing base (a77de95) to head (072ab1a).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2195      +/-   ##
============================================
- Coverage     45.95%   45.17%   -0.79%     
- Complexity      797      811      +14     
============================================
  Files            65       67       +2     
  Lines          3066     3119      +53     
============================================
  Hits           1409     1409              
- Misses         1657     1710      +53     

@susnux
Copy link
Collaborator

susnux commented Jun 1, 2024

CI is failing a lot (PHP code style, missing stubs for psalm, missing unit tests), please fix this :)

Also cc @Chartman123, what do you think about this?

@Chartman123
Copy link
Collaborator

@susnux if this doesn't come with any problems when the Analytics app isn't installed, I'd be fine with it.

@Rello
Copy link
Author

Rello commented Jun 4, 2024

@Chartman123 I just did another test on my side - all clean. no nextcloud.log
happy for an other retest

...we only need to find a way for the psalm and the unit-tests...

@susnux susnux force-pushed the Rello-Analytics branch 2 times, most recently from 41fa849 to 2f4feae Compare June 12, 2024 21:25
Copy link
Contributor

@Fenn-CS Fenn-CS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look okay.

psalm.xml Outdated Show resolved Hide resolved
Rello and others added 10 commits June 13, 2024 12:02
Signed-off-by: Rello <Rello@users.noreply.github.com>
Signed-off-by: Rello <Rello@users.noreply.github.com>
Signed-off-by: Rello <Rello@users.noreply.github.com>
Signed-off-by: Rello <Rello@users.noreply.github.com>
Signed-off-by: Rello <Rello@users.noreply.github.com>
Signed-off-by: Rello <Rello@users.noreply.github.com>
Signed-off-by: Rello <Rello@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Rello and others added 3 commits June 13, 2024 17:32
Co-authored-by: F. E Noel Nfebe <fenn25.fn@gmail.com>
Signed-off-by: Rello <Rello@users.noreply.github.com>
Signed-off-by: Rello <Rello@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants