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

fix: compute subtotals/totals for boolean types (DHIS2-9155) #3194

Merged
merged 17 commits into from
Oct 18, 2024

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Aug 28, 2024

Fixes DHIS2-9155

Requires dhis2/analytics#1696 and dhis2/analytics#1700


Key features

  1. bump analytics to 26.8.7
  2. add Cypress tests for totals/cumulative options with mixed valueTypes/totalAggregationTypes along rows

Description

The actual fix is in analytics.
See dhis2/analytics#1696 and dhis2/analytics#1700 for more details.


TODO

  • Cypress tests
  • Update docs
  • Manual testing

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Aug 28, 2024

🚀 Deployed on https://pr-3194.data-visualizer.netlify.dhis2.org

@dhis2-bot dhis2-bot temporarily deployed to netlify August 28, 2024 13:17 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify September 3, 2024 09:25 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify September 16, 2024 12:05 Inactive
@edoardo edoardo added the e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud label Sep 21, 2024
Copy link

cypress bot commented Sep 21, 2024

Data Visualizer App    Run #2929

Run Properties:  status check passed Passed #2929  •  git commit d39e456e0e ℹ️: Merge 473c67988e27655d1faa61d76086e0f82d2d0af0 into 4e32ed0998f0cd837b5525946e6f...
Project Data Visualizer App
Branch Review fix/totals-for-boolean-types-DHIS2-9155
Run status status check passed Passed #2929
Run duration 09m 58s
Commit git commit d39e456e0e ℹ️: Merge 473c67988e27655d1faa61d76086e0f82d2d0af0 into 4e32ed0998f0cd837b5525946e6f...
Committer Edoardo Sabadelli
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 2
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 745
View all changes introduced in this branch ↗︎

@edoardo edoardo removed the e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud label Sep 21, 2024
@dhis2-bot dhis2-bot temporarily deployed to netlify October 9, 2024 14:38 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 10, 2024 09:07 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 11, 2024 08:44 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 11, 2024 19:34 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 15, 2024 09:43 Inactive
Since cli-app-scripts has already been updated to a version with the new
plugin components, charts and PT have problems in dashboard because they
still use the postRobot communication internally and the
dashboard version does not use the new platform components.
The skipPluginLogic flag avoids the issues by disabling the whole
implementation in the platform plugin components at build time.
@dhis2-bot dhis2-bot temporarily deployed to netlify October 18, 2024 06:48 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 18, 2024 08:15 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 18, 2024 08:31 Inactive
Copy link
Collaborator

@janhenrikoverland janhenrikoverland left a comment

Choose a reason for hiding this comment

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

Tested here: dhis2/analytics#1700 (review)

Approved when:

  • Version is updated
  • Cypress tests for cumulative values are added

@dhis2-bot dhis2-bot temporarily deployed to netlify October 18, 2024 14:30 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify October 18, 2024 14:33 Inactive
@edoardo edoardo merged commit ebc0806 into master Oct 18, 2024
20 checks passed
@edoardo edoardo deleted the fix/totals-for-boolean-types-DHIS2-9155 branch October 18, 2024 16:24
dhis2-bot added a commit that referenced this pull request Oct 18, 2024
## [100.8.1](v100.8.0...v100.8.1) (2024-10-18)

### Bug Fixes

* compute subtotals/totals for boolean types (DHIS2-9155) ([#3194](#3194)) ([ebc0806](ebc0806))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.8.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants