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: use interpretations component from Analytics (DHIS2-15441) #2430

Merged
merged 30 commits into from
Mar 4, 2024

Conversation

jenniferarnesen
Copy link
Collaborator

@jenniferarnesen jenniferarnesen commented Jun 21, 2023

Fixes: https://dhis2.atlassian.net/browse/DHIS2-15441
Depends on: dhis2/analytics#1510
Depends on: dhis2/maps-app#2936
Depends on: dhis2/maps-app#2937

Replace the old d2-ui interpretations component with the one in analytics used by DV and LL. With this change, the old d2-ui sharing dialog is also replaced with the new one from ui.

Screenshots:
New:
image
Old:
image

Looking at one interpretation:
New:
image
Old:
image

Two old features were removed:

  • Show previous interpretations button. This button toggled whether max 5 interpretations are shown, or all are shown.
    image
  • Replying to a reply. This is not even supported by the backend as all replies are connected directly with the original interpretation, so the reply button on a Reply never made sense anyway.
    image

zip file size improvements:
Old zip file: 5057221
New zip file: 3575577 (29% smaller)

chunks (build/static/js/xyz.js)
Old
3053115 Jun 23 13:58 8877.6fc0d050.chunk.js

New:
1665003 Jun 23 13:55 396.371a423b.chunk.js (45% smaller)

Old source-map-explorer for dependencies bundle (all the dependencies in red were only there bc of the old interpretations component):
image

New source-map-explorer for dependencies bundle (all those red dependencies were removed):
image

jenniferarnesen and others added 10 commits June 7, 2023 14:23
The verify workflow should run on tags, so that a corresponding branch is created
in the d2-ci repository

Do not run the release step on PRs, but do allow it for pushes

yarn cypress install sometimes fails. Turns out it isn't necessary anyway
since the cypress-io gh action takes care of installing cypress.
Bumps [json5](https://github.com/json5/json5) from 1.0.1 to 1.0.2.
- [Release notes](https://github.com/json5/json5/releases)
- [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md)
- [Commits](json5/json5@v1.0.1...v1.0.2)

---
updated-dependencies:
- dependency-name: json5
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jen Jones Arnesen <jennifer@dhis2.org>
* chore: nightly failing to set up cypress (#2378)

The verify workflow should run on tags, so that a corresponding branch is created
in the d2-ci repository

Do not run the release step on PRs, but do allow it for pushes

yarn cypress install sometimes fails. Turns out it isn't necessary anyway
since the cypress-io gh action takes care of installing cypress.

* fix(translations): sync translations from transifex (dev)

Automatically merged.

* docs: move docs to app (#2384)

* chore(release): cut 100.0.1 [skip release]

## [100.0.1](v100.0.0...v100.0.1) (2023-06-12)

### Bug Fixes

* **translations:** sync translations from transifex (dev) ([5b0b3b6](5b0b3b6))

* chore(deps): bump @dhis2/analytics from 25.1.10 to 25.1.15

Bumps [@dhis2/analytics](https://github.com/dhis2/analytics) from 25.1.10 to 25.1.15.
- [Release notes](https://github.com/dhis2/analytics/releases)
- [Changelog](https://github.com/dhis2/analytics/blob/master/CHANGELOG.md)
- [Commits](dhis2/analytics@v25.1.10...v25.1.15)

---
updated-dependencies:
- dependency-name: "@dhis2/analytics"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Jen Jones Arnesen <jennifer@dhis2.org>
Co-authored-by: @dhis2-bot <apps@dhis2.org>
Co-authored-by: Jan Henrik Øverland <janhenrik.overland@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@jenniferarnesen jenniferarnesen marked this pull request as draft June 21, 2023 09:24
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

jenniferarnesen and others added 9 commits June 27, 2023 11:15
* chore: nightly failing to set up cypress (#2378)

The verify workflow should run on tags, so that a corresponding branch is created
in the d2-ci repository

Do not run the release step on PRs, but do allow it for pushes

yarn cypress install sometimes fails. Turns out it isn't necessary anyway
since the cypress-io gh action takes care of installing cypress.

* fix(translations): sync translations from transifex (dev)

Automatically merged.

* docs: move docs to app (#2384)

* chore(release): cut 100.0.1 [skip release]

## [100.0.1](v100.0.0...v100.0.1) (2023-06-12)

### Bug Fixes

* **translations:** sync translations from transifex (dev) ([5b0b3b6](5b0b3b6))

* chore(deps): bump @dhis2/ui from 8.13.6 to 8.13.11

Bumps [@dhis2/ui](https://github.com/dhis2/ui/tree/HEAD/collections/ui) from 8.13.6 to 8.13.11.
- [Release notes](https://github.com/dhis2/ui/releases)
- [Changelog](https://github.com/dhis2/ui/blob/master/CHANGELOG.md)
- [Commits](https://github.com/dhis2/ui/commits/v8.13.11/collections/ui)

---
updated-dependencies:
- dependency-name: "@dhis2/ui"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Jen Jones Arnesen <jennifer@dhis2.org>
Co-authored-by: @dhis2-bot <apps@dhis2.org>
Co-authored-by: Jan Henrik Øverland <janhenrik.overland@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This is temporary during summer 2023 when there is very little frontend or backend activity
…his2/dashboard-app into feat/replace-interpretations-component
@jenniferarnesen jenniferarnesen added the on hold Do not review or merge yet label Aug 31, 2023
@jenniferarnesen jenniferarnesen changed the title fix: replace interpretations component fix: use interpretations component from Analytics (DHIS2-15441) Sep 27, 2023
@jenniferarnesen jenniferarnesen marked this pull request as draft September 27, 2023 14:13
@edoardo edoardo marked this pull request as ready for review March 4, 2024 09:15
@dhis2-bot
Copy link
Contributor

dhis2-bot commented Mar 4, 2024

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

@dhis2-bot dhis2-bot temporarily deployed to netlify March 4, 2024 09:39 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify March 4, 2024 10:03 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify March 4, 2024 10:52 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify March 4, 2024 14:01 Inactive
@edoardo edoardo merged commit 3550cfa into dev Mar 4, 2024
14 checks passed
@edoardo edoardo deleted the feat/replace-interpretations-component branch March 4, 2024 14:28
@jenniferarnesen jenniferarnesen removed the on hold Do not review or merge yet label Mar 7, 2024
dhis2-bot added a commit that referenced this pull request Mar 21, 2024
# [100.1.0](v100.0.4...v100.1.0) (2024-03-21)

### Bug Fixes

* show tag/tooltip about not applied filters for Outlier table (DHIS2-13858) ([#2937](#2937)) ([dffb754](dffb754))
* show tag/tooltip only when a filter other than ou/pe is used ([#2941](#2941)) ([b6b8850](b6b8850))
* **translations:** sync translations from transifex (dev) ([755c6df](755c6df))
* use interpretations component from Analytics (DHIS2-15441) ([#2430](#2430)) ([0af5c5b](0af5c5b))
* **translations:** sync translations from transifex (dev) ([0de3606](0de3606))
* **translations:** sync translations from transifex (dev) ([b88e87c](b88e87c))
* **translations:** sync translations from transifex (dev) ([#2920](#2920)) ([add7139](add7139))

### Features

* support outlier table plugin (DHIS2-16751) ([#2912](#2912)) ([1af22d5](1af22d5))
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.

5 participants