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: report view as MAP_VIEW not VISUALIZATION_VIEW (DHIS2-16250) #3086

Merged
merged 2 commits into from
Dec 20, 2023

Conversation

jenniferarnesen
Copy link
Collaborator

Fixes https://dhis2.atlassian.net/browse/DHIS2-16250

The view count should still be correct in the Interpretations and Details panel.

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Dec 19, 2023

🚀 Deployed on https://pr-3086--dhis2-maps.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify December 19, 2023 12:27 Inactive
Copy link

cypress bot commented Dec 19, 2023

4 flaky tests on run #2811 ↗︎

0 45 16 0 Flakiness 4

Details:

Merge 1f4d99f into 0fee930...
Project: maps Commit: 88a90d4804 ℹ️
Status: Passed Duration: 03:16 💡
Started: Dec 20, 2023 5:31 PM Ended: Dec 20, 2023 5:35 PM
Flakiness  layers/thematiclayer.cy.js • 1 flaky test • e2e-chrome-parallel-2.40

View Output

Test Artifacts
Thematic Layers > adds a thematic layer for OU Bombali Screenshots
Flakiness  systemsettings.cy.js • 1 flaky test • e2e-chrome-parallel-2.40

View Output

Test Artifacts
systemSettings > includes Weekly period type when weekly periods not hidden in system settings Screenshots
Flakiness  dataDownload.cy.js • 1 flaky test • e2e-chrome-parallel-2.40

View Output

Test Artifacts
Data Download > downloads data from a thematic layer Screenshots
Flakiness  interpretations.cy.js • 1 flaky test • e2e-chrome-parallel-2.40

View Output

Test Artifacts
Interpretations > view interpretation after creating a map Screenshots

Review all test suite changes for PR #3086 ↗︎

@dhis2-bot dhis2-bot temporarily deployed to netlify December 20, 2023 17:21 Inactive
@jenniferarnesen jenniferarnesen merged commit 008e7fe into dev Dec 20, 2023
15 checks passed
@jenniferarnesen jenniferarnesen deleted the fix/map-view-not-vis-view branch December 20, 2023 17:37
@cypress cypress bot mentioned this pull request Dec 20, 2023
@janhenrikoverland janhenrikoverland changed the title fix: report view as MAP_VIEW not VISUALIZATION_VIEW fix: report view as MAP_VIEW not VISUALIZATION_VIEW (DHIS2-16250) Dec 21, 2023
dhis2-bot added a commit that referenced this pull request Dec 21, 2023
## [100.2.3](v100.2.2...v100.2.3) (2023-12-21)

### Bug Fixes

* conditionally set PR title in dhis2-verify-app workflow ([#3089](#3089)) ([03eed2d](03eed2d))
* report as MAP_VIEW not VISUALIZATION_VIEW ([#3086](#3086)) ([008e7fe](008e7fe))
* **translations:** sync translations from transifex (dev) ([dcf7743](dcf7743))
* **translations:** sync translations from transifex (dev) ([7d98a8a](7d98a8a))
* **translations:** sync translations from transifex (dev) ([6e8b82b](6e8b82b))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.2.3 🎉

The release is available on:

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