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: unclear 'show no data' #3204

Merged
merged 16 commits into from
Jun 7, 2024
Merged

fix: unclear 'show no data' #3204

merged 16 commits into from
Jun 7, 2024

Conversation

BRaimbault
Copy link
Collaborator

@BRaimbault BRaimbault commented May 15, 2024

Implement DHIS2-15799
Linked maps-gl PR: dhis2/maps-gl#573


Update text "Show no data" in NoDataColor component.
Add logic to display "no data" text if value is missing in Popup component.


Before:

Screenshare.-.2024-05-15.2_37_54.PM.mp4

After: ("no data" has been changed to "No data")

Screenshare.-.2024-05-16.2_03_10.AM.mp4

@dhis2-bot
Copy link
Contributor

dhis2-bot commented May 15, 2024

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

@dhis2-bot dhis2-bot temporarily deployed to netlify May 15, 2024 12:43 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify May 15, 2024 12:51 Inactive
Copy link

cypress bot commented May 15, 2024

6 flaky tests on run #3239 ↗︎

0 118 32 0 Flakiness 6

Details:

Merge 6a30ebb into 230dc08...
Project: maps Commit: 986b2fa43e ℹ️
Status: Passed Duration: 05:34 💡
Started: Jun 4, 2024 9:30 AM Ended: Jun 4, 2024 9:36 AM
Flakiness  routes.cy.js • 1 flaky test • e2e-chrome-parallel-2.41

View Output

Test Artifacts
Routes > loads download page currentAnalyticalObject (hash) Screenshots
Flakiness  dataDownload.cy.js • 1 flaky test • e2e-chrome-parallel-2.41

View Output

Test Artifacts
Data Download > downloads data from an event layer Screenshots
Flakiness  interpretations.cy.js • 1 flaky test • e2e-chrome-parallel-2.41

View Output

Test Artifacts
Interpretations > view interpretation after creating a map Screenshots
Flakiness  routes.cy.js • 1 flaky test • e2e-chrome-parallel-2.40

View Output

Test Artifacts
Routes > loads download page currentAnalyticalObject (hash) 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

The first 5 flaky specs are shown, see all 6 specs in Cypress Cloud.

Review all test suite changes for PR #3204 ↗︎

@jenniferarnesen jenniferarnesen marked this pull request as draft May 16, 2024 07:32
@jenniferarnesen jenniferarnesen marked this pull request as ready for review May 16, 2024 07:32
@dhis2-bot dhis2-bot temporarily deployed to netlify May 23, 2024 14:12 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 3, 2024 15:18 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 3, 2024 15:30 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 4, 2024 09:18 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 4, 2024 12:46 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 4, 2024 12:57 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 6, 2024 11:30 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 6, 2024 13:58 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 6, 2024 14:04 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 7, 2024 10:44 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 7, 2024 11:34 Inactive
@BRaimbault BRaimbault merged commit ec24ca0 into dev Jun 7, 2024
21 checks passed
@BRaimbault BRaimbault deleted the fix/DHIS2-15799 branch June 7, 2024 12:07
dhis2-bot added a commit that referenced this pull request Jun 13, 2024
## [100.5.5](v100.5.4...v100.5.5) (2024-06-13)

### Bug Fixes

* **translations:** sync translations from transifex (dev) ([#3237](#3237)) ([8e9a46d](8e9a46d))
* unclear 'show no data' ([#3204](#3204)) ([ec24ca0](ec24ca0))
* use improved rich text editor from analytics (DHIS2-15522) ([#3236](#3236)) ([3c842d4](3c842d4))
* **translations:** sync translations from transifex (dev) ([230dc08](230dc08))
* **translations:** sync translations from transifex (dev) ([83393cb](83393cb))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.5.5 🎉

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
Development

Successfully merging this pull request may close these issues.

None yet

3 participants