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

chore: stop fetching schemas and replace d2 with dataEngine #3056

Merged
merged 14 commits into from
Aug 26, 2024

Conversation

janhenrikoverland
Copy link
Collaborator

@janhenrikoverland janhenrikoverland commented Aug 16, 2024

The schemas aren't needed anymore.
Also, replaced d2 with dataEngine:

  • post dataStatistics
  • getting currentUser
  • getting list of installed apps

We have to unfortunately disable some checks in the dashboard filter tests because cypress seems to be having a fit about the iframes. You could argue that the tests shouldn't be testing the content of the plugins anyway since we don't have control over which versions of the plugins are installed. The plugins should be getting tested in their respective apps. Still, it is sad to have to disable the checks.

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Aug 16, 2024

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

@dhis2-bot dhis2-bot temporarily deployed to netlify August 16, 2024 08:32 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 19, 2024 11:21 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 19, 2024 14:18 Inactive
@jenniferarnesen jenniferarnesen changed the title chore: remove d2 chore: stop fetching schemas and replace d2 with dataEngine Aug 19, 2024
@jenniferarnesen jenniferarnesen added the e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud label Aug 20, 2024
Copy link

cypress bot commented Aug 20, 2024

dashboards-app    Run #4980

Run Properties:  status check passed Passed #4980  •  git commit 7aff5d0f36 ℹ️: Merge 69c7a57b673e3703321037bc9aa057a7e82b1257 into 9fe6ae524e965df083e6b6cacce2...
Project dashboards-app
Branch Review chore/remove-unneeded-schemas
Run status status check passed Passed #4980
Run duration 02m 07s
Commit git commit 7aff5d0f36 ℹ️: Merge 69c7a57b673e3703321037bc9aa057a7e82b1257 into 9fe6ae524e965df083e6b6cacce2...
Committer Jan Henrik Øverland
View all properties for this run ↗︎

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

@dhis2-bot dhis2-bot temporarily deployed to netlify August 21, 2024 17:41 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 21, 2024 17:50 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 22, 2024 08:44 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 23, 2024 07:57 Inactive
Copy link
Collaborator Author

@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.

@dhis2-bot dhis2-bot temporarily deployed to netlify August 23, 2024 14:00 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 23, 2024 14:53 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify August 26, 2024 12:16 Inactive
@jenniferarnesen jenniferarnesen merged commit 49428c6 into master Aug 26, 2024
43 checks passed
@jenniferarnesen jenniferarnesen deleted the chore/remove-unneeded-schemas branch August 26, 2024 12:38
dhis2-bot added a commit that referenced this pull request Aug 26, 2024
## [100.2.4](v100.2.3...v100.2.4) (2024-08-26)

### Bug Fixes

* add missing dependencies for semantic-release ([#3062](#3062)) ([9fe6ae5](9fe6ae5))
* display correct app version in dhis2 and updated workflows ([#3061](#3061)) ([69b8425](69b8425))
* stop fetching schemas and replace d2 with dataEngine ([#3056](#3056)) ([49428c6](49428c6))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.2.4 🎉

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
e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants