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 #9075. Handle cases when domain isnot in format start--end #9076

Merged
merged 2 commits into from
Apr 6, 2023

Conversation

offtherailz
Copy link
Member

Description

This PR fixes #9075 handling the cases when DescribeDomain does not return "start--end" format.

  • No data: empty string (range is set as for saved data)
  • 1 value only: only current time is set, range is set to default interval
  • list of values, getting start and end value to set the range.

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?

#9075
What is the new behavior?

Timeline do not crash. Current value is not null.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

@offtherailz offtherailz added this to the 2023.01.01 milestone Apr 5, 2023
@offtherailz offtherailz requested a review from dsuren1 April 5, 2023 16:25
@offtherailz offtherailz self-assigned this Apr 5, 2023
@offtherailz offtherailz added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Apr 5, 2023
Copy link
Contributor

@dsuren1 dsuren1 left a comment

Choose a reason for hiding this comment

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

LGTM!

web/client/epics/timeline.js Outdated Show resolved Hide resolved
@dsuren1 dsuren1 merged commit 5e3c57e into geosolutions-it:master Apr 6, 2023
@dsuren1
Copy link
Contributor

dsuren1 commented Apr 6, 2023

@ElenaGallo Kindly test it in Dev and let us know if it can be backported. Thanks!

@offtherailz
Copy link
Member Author

Going to backport it.

@offtherailz
Copy link
Member Author

@ElenaGallo about testing. I will take care of testing it in the project (I already did) because this bug happens in very special cases.
Only tests for timeline are enough

offtherailz added a commit to offtherailz/MapStore2 that referenced this pull request Apr 6, 2023
…art--end (geosolutions-it#9076)

* Fix geosolutions-it#9075. Handle cases when domain isnot in format start--end

* Update web/client/epics/timeline.js

---------

Co-authored-by: Suren <[email protected]>
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Apr 6, 2023
tdipisa pushed a commit that referenced this pull request Apr 6, 2023
#9079)

* Fix #9075. Handle cases when domain isnot in format start--end

* Update web/client/epics/timeline.js

---------

Co-authored-by: Suren <[email protected]>
@ElenaGallo
Copy link
Contributor

Test passed, @offtherailz it is possible to backport this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Timeline crash if time layer has few data
4 participants