forms
forms copied to clipboard
Data source for the Analytics App
This pull adds a data source to report on forms with Analyitcs
- [x] real time reporting
- [x] scheduled data loads (for trends)
- [x] charts and/or tables
- [x] multi page dashboards (coming with Analytics 5)
- [x] PDF/print dashboards (coming with Analytics 5)
- [ ] one-click dashboard creation for all questions of a form
Data source selection
Single report
Dashboard
Codecov Report
Attention: Patch coverage is 0% with 53 lines in your changes missing coverage. Please review.
Project coverage is 45.17%. Comparing base (
e019081) to head (0530cd1). Report is 14 commits behind head on main.
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
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?
@susnux if this doesn't come with any problems when the Analytics app isn't installed, I'd be fine with it.
@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...
@Rello are you ready with this PR? Perhaps you could add some more unit tests, so that the Codecov workflows turn green, too 🙂
Before we merge this, also please do a rebase on current main and squash all the commits into one single commit.
Hi @Chartman123 I will retest one more time and post an update here. regarding unit tests, I would need assistant. I never did these in the past as the dev-capacity is pretty limited...
@Rello please don't merge the main branch into your branch and use git rebase instead :)
@Rello please don't merge the main branch into your branch and use
git rebaseinstead :)
Hi, any hint on the background? I was always doing this until now. but open to learn something better. Tx!
Rebasing results in a cleaner commit history, when we merge the changes back into our main :)
@Chartman123 @susnux all tests are green - besides the codecov part and I have no idea how to solve this. what do you suggest to do here?
🙏🏻
@susnux we should have squashed the commits into one single commit first... ;)