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.

Sign up for GitHub

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

Jump to bottom

fix #9601 styles #9632

Merged
merged 1 commit into from
Oct 20, 2023
Merged

fix #9601 styles #9632

merged 1 commit into from
Oct 20, 2023

Conversation

MV88
Copy link
Contributor

@MV88 MV88 commented Oct 19, 2023

Description

Fixing the style of the line which was not being properly handled

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?

fix #9601

What is the new behavior?

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

@MV88 MV88 added BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch C265-ATOLCD-2023-DEV_MS_CNR labels Oct 19, 2023
@MV88 MV88 requested a review from offtherailz October 19, 2023 16:12
@MV88 MV88 self-assigned this Oct 19, 2023
@MV88 MV88 mentioned this pull request Oct 19, 2023
Merged
12 tasks
offtherailz
offtherailz approved these changes Oct 20, 2023
View reviewed changes
Copy link
Member

@offtherailz offtherailz left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM. I didn't tested, but the changes looks correct.

@tdipisa tdipisa added this to the 2023.02.01 milestone Oct 20, 2023
@tdipisa tdipisa merged commit 05754ef into geosolutions-it:master Oct 20, 2023
4 checks passed
@tdipisa
Copy link
Member

tdipisa commented Oct 20, 2023

@ElenaGallo please test and let us know for the backport.

MV88 added a commit to MV88/MapStore2 that referenced this pull request Oct 20, 2023
@MV88
fix geosolutions-it#9601 styles (geosolutions-it#9632)
3e66b56
@MV88 MV88 mentioned this pull request Oct 20, 2023
Merged
MV88 added a commit that referenced this pull request Oct 20, 2023
@MV88
fix #9601 styles (#9632) (#9637)
f30c5c7
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@offtherailz offtherailz offtherailz approved these changes

Assignees

@MV88 MV88

Labels
C265-ATOLCD-2023-DEV_MS_CNR
Projects
None yet
Milestone
2023.02.01
Development

Successfully merging this pull request may close these issues.

GeoProcessing - it does not includes points style that can result from intersection of polygon and point
3 participants
@MV88 @tdipisa @offtherailz