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

refactor: use shared dashboard plugin wrapper #396

Draft
wants to merge 36 commits into
base: master
Choose a base branch
from

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Jun 21, 2023

Part of the fix for: DHIS2-17268

Requires dhis2/analytics#1672


Key features

  1. use shared dashboard plugin wrapper

Description

Generic components for plugins are available in app-platform and app-runtime.
Analytics apps are currently having the same components replicated in each repo, a shared dashboard plugin wrapper has been added in analytics.


Known issues

  • an infinite loop happened and this seems to solve it, I'm not sure why and if this is a proper solution.

Screenshots

Screenshot 2023-06-21 at 16 25 33

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Jun 21, 2023

🚀 Deployed on https://pr-396.line-listing.netlify.dhis2.org

@dhis2-bot dhis2-bot temporarily deployed to netlify June 21, 2023 14:41 Inactive
@edoardo edoardo force-pushed the refactor/use-platform-plugin-components branch from b365999 to e156657 Compare October 9, 2023 08:40
@edoardo edoardo force-pushed the refactor/use-platform-plugin-components branch from 795334d to 16060c2 Compare May 24, 2024 13:21
@dhis2-bot dhis2-bot temporarily deployed to netlify May 24, 2024 13:23 Inactive
@edoardo edoardo force-pushed the refactor/use-platform-plugin-components branch from 16060c2 to ea99422 Compare May 27, 2024 13:43
@dhis2-bot dhis2-bot temporarily deployed to netlify May 27, 2024 13:46 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify May 29, 2024 13:49 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify May 30, 2024 12:59 Inactive
@edoardo edoardo force-pushed the refactor/use-platform-plugin-components branch from dc5c207 to af32223 Compare June 4, 2024 13:37
@dhis2-bot dhis2-bot temporarily deployed to netlify June 4, 2024 13:47 Inactive
@edoardo edoardo changed the title refactor: use platform/runtime plugin components refactor: use shared dashboard plugin wrapper Jun 4, 2024
@edoardo edoardo force-pushed the refactor/use-platform-plugin-components branch from af32223 to 3ed21de Compare July 1, 2024 14:11
@dhis2-bot dhis2-bot temporarily deployed to netlify July 1, 2024 14:13 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 3, 2024 09:58 Inactive
@edoardo edoardo added the e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud label Jul 4, 2024
Copy link

cypress bot commented Jul 4, 2024



Test summary

579 9 0 0Flakiness 0


Run details

Project line-listing-app
Status Failed
Commit f599dcf ℹ️
Started Aug 19, 2024 9:34 AM
Ended Aug 19, 2024 9:51 AM
Duration 16:54 💡
OS Linux Ubuntu -
Browser Chrome 127

View run in Cypress Cloud ➡️


Failures

Run group: e2e-chrome-parallel-2.39-2 (Linux, Chrome )
createEnrollment.cy.js Failed
1 enrollment > moves a dimension to filter
Run group: e2e-chrome-parallel-2.39-5 (Linux, Chrome )
conditions/optionSetCondition.cy.js Failed
1 Option set condition > Options with same code but from different option sets display correctly
Run group: e2e-chrome-parallel-2.39-1 (Linux, Chrome )
orgUnitDimension.cy.js Failed
1 Org unit dimension > saves and loads items correctly
Run group: e2e-chrome-parallel-2.40-1 (Linux, Chrome )
orgUnitDimension.cy.js Failed
1 Org unit dimension > saves and loads items correctly
Run group: e2e-chrome-parallel-2.40-2 (Linux, Chrome )
createEnrollment.cy.js Failed
1 enrollment > moves a dimension to filter
Run group: e2e-chrome-parallel-2.40-5 (Linux, Chrome )
conditions/optionSetCondition.cy.js Failed
1 Option set condition > Options with same code but from different option sets display correctly
Run group: e2e-chrome-parallel-2.41-1 (Linux, Chrome )
orgUnitDimension.cy.js Failed
1 Org unit dimension > saves and loads items correctly
Run group: e2e-chrome-parallel-2.41-2 (Linux, Chrome )
createEnrollment.cy.js Failed
1 enrollment > moves a dimension to filter
Run group: e2e-chrome-parallel-2.41-5 (Linux, Chrome )
conditions/optionSetCondition.cy.js Failed
1 Option set condition > Options with same code but from different option sets display correctly

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud

@dhis2-bot dhis2-bot temporarily deployed to netlify July 4, 2024 13:03 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 4, 2024 14:24 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 11, 2024 13:01 Inactive
Copy link
Contributor

@HendrikThePendric HendrikThePendric left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

src/components/Visualization/Visualization.js Outdated Show resolved Hide resolved
src/components/Visualization/Visualization.js Show resolved Hide resolved
src/components/Visualization/Visualization.js Outdated Show resolved Hide resolved
@dhis2-bot dhis2-bot temporarily deployed to netlify July 12, 2024 08:16 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 12, 2024 12:13 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 30, 2024 08:54 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 30, 2024 14:02 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 31, 2024 07:57 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 31, 2024 08:37 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify July 31, 2024 10:15 Inactive
@edoardo edoardo force-pushed the refactor/use-platform-plugin-components branch from a53dde3 to 8633759 Compare August 13, 2024 13:53
@dhis2-bot dhis2-bot temporarily deployed to netlify August 15, 2024 13:51 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 15, 2024 14:04 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 16, 2024 08:22 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 16, 2024 09:51 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 19, 2024 09:30 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants