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: ou group filter crashes in custom widgets [v39] #2692

Merged
merged 3 commits into from
Sep 14, 2023

Conversation

jenniferarnesen
Copy link
Collaborator

@jenniferarnesen jenniferarnesen commented Sep 6, 2023

backport of #2691

Fixes DHIS2-14544

If path exists, then parse it, otherwise use id (which will be the case for groups, levels and USER_ORGUNITS)

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Sep 6, 2023

🚀 Deployed on https://pr-2692--dhis2-dashboard.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify September 6, 2023 08:49 Inactive
@cypress
Copy link

cypress bot commented Sep 6, 2023

8 failed tests on run #4094 ↗︎

8 0 0 51 Flakiness 0

Details:

Merge branch 'v39' into fix/custom-ou-group-filter-crash-v39
Project: dashboards-app Commit: bd0c0b04af
Status: Failed Duration: 01:46 💡
Started: Sep 13, 2023 11:08 AM Ended: Sep 13, 2023 11:10 AM
Failed  view/view_dashboard.feature • 1 failed test • e2e

View Output Video

Test Artifacts
Viewing dashboards > I switch between dashboards Output Screenshots Video
Failed  view/view_errors.feature • 1 failed test • e2e

View Output Video

Test Artifacts
Errors while in view mode > There are no dashboards Output Screenshots Video
Failed  view/responsive_dashboard.feature • 1 failed test • e2e

View Output Video

Test Artifacts
Small screen dashboard > I view a dashboard Output Screenshots Video
Failed  edit/edit_dashboard.feature • 1 failed test • e2e

View Output Video

Test Artifacts
Creating, editing and deleting dashboard > I create a new dashboard Output Screenshots Video
Failed  view/dashboard_filter.feature • 1 failed test • e2e

View Output Video

Test Artifacts
Dashboard filter > I add a Period filter Output Screenshots Video

The first 5 failed specs are shown, see all 8 specs in Cypress Cloud.

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@dhis2-bot dhis2-bot temporarily deployed to netlify September 13, 2023 11:07 Inactive
@jenniferarnesen jenniferarnesen merged commit 6c3b3d6 into v39 Sep 14, 2023
11 checks passed
@jenniferarnesen jenniferarnesen deleted the fix/custom-ou-group-filter-crash-v39 branch September 14, 2023 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants