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: set period for each split view map #2721

Merged
merged 4 commits into from
Jun 27, 2023
Merged

fix: set period for each split view map #2721

merged 4 commits into from
Jun 27, 2023

Conversation

turban
Copy link
Contributor

@turban turban commented Jun 26, 2023

This PR makes sure each new split view map gets the correct period when user is drilling down/up.

Fixes: https://dhis2.atlassian.net/browse/DHIS2-15437

After this PR the different periods are kept when drilling:
split-view-drilling

The PR also fixes some console errors when we try to update a map that has been removed due to user interaction.

Screenshot 2023-06-26 at 13 41 13

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Jun 26, 2023

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

@dhis2-bot dhis2-bot temporarily deployed to netlify June 26, 2023 12:05 Inactive
@turban turban requested a review from jenniferarnesen June 26, 2023 12:06
@cypress
Copy link

cypress bot commented Jun 26, 2023

Passing run #2200 ↗︎

0 33 20 0 Flakiness 0

Details:

Merge feea156 into 3ba7831...
Project: maps Commit: 3f938c81e6 ℹ️
Status: Passed Duration: 03:47 💡
Started: Jun 27, 2023 12:53 PM Ended: Jun 27, 2023 12:56 PM

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 June 27, 2023 12:45 Inactive
@turban turban merged commit 32578ec into dev Jun 27, 2023
@turban turban deleted the fix/DHIS2-15437 branch June 27, 2023 13:11
@cypress cypress bot mentioned this pull request Jun 27, 2023
dhis2-bot added a commit that referenced this pull request Aug 31, 2023
## [100.1.6](v100.1.5...v100.1.6) (2023-08-31)

### Bug Fixes

* **translations:** sync translations from transifex (dev) ([3a3b81e](3a3b81e))
* **translations:** sync translations from transifex (dev) ([3effb0d](3effb0d))
* keep layer visibility when period is changed ([#2705](#2705)) ([df32ea4](df32ea4))
* set period for each split view map ([#2721](#2721)) ([6ec0eba](6ec0eba))
* Set rendering strategy to single if not relative period ([#2703](#2703)) ([c47484b](c47484b))
* upgrade landcover dataset ([#2732](#2732)) ([3e93ced](3e93ced))
* **translations:** sync translations from transifex (dev) ([22d43c7](22d43c7))
* **translations:** sync translations from transifex (dev) ([b5da40a](b5da40a))
* **translations:** sync translations from transifex (dev) ([a090132](a090132))
* **translations:** sync translations from transifex (dev) ([21acc31](21acc31))
* **translations:** sync translations from transifex (dev) ([4597cbb](4597cbb))
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