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

[Backport 2023.01.xx] #9036: Fix - Catalog mode and result retention (#9037) #9045

Merged
merged 1 commit into from
Mar 23, 2023
Merged

[Backport 2023.01.xx] #9036: Fix - Catalog mode and result retention (#9037) #9045

merged 1 commit into from
Mar 23, 2023

Conversation

dsuren1
Copy link
Contributor

@dsuren1 dsuren1 commented Mar 22, 2023

#9036: Fix - Catalog mode and result retention (#9037)

@dsuren1 dsuren1 added this to the 2023.01.01 milestone Mar 22, 2023
@dsuren1 dsuren1 requested a review from MV88 March 22, 2023 13:32
@dsuren1 dsuren1 self-assigned this Mar 22, 2023
@dsuren1 dsuren1 linked an issue Mar 22, 2023 that may be closed by this pull request
Catalog issue when opening with a service other than the previous map #9036
Closed
1 task
tdipisa
tdipisa reviewed Mar 22, 2023
View reviewed changes
Copy link
Member

@tdipisa tdipisa left a comment

Choose a reason for hiding this comment

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

@dsuren1 why changes to the other two filed in the original PR are not necessary there?

@dsuren1
Copy link
Contributor Author

dsuren1 commented Mar 23, 2023 •
edited

@tdipisa
Those files changes are from the commit which was replaced by the original PR of this backport, in the dev branch which is not needed in stable branch, hence we see only two files modified in the backport

@dsuren1 dsuren1 requested a review from tdipisa March 23, 2023 06:29
@MV88 MV88 added the Backport label Mar 23, 2023
@MV88 MV88 merged commit cf943ea into geosolutions-it:2023.01.xx Mar 23, 2023
@MV88
Copy link
Contributor

MV88 commented Mar 23, 2023

@ElenaGallo please Test it in QA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MV88 MV88 MV88 approved these changes

@tdipisa tdipisa Awaiting requested review from tdipisa

Assignees

@dsuren1 dsuren1

Labels
Backport bug Internal
Projects
None yet
Milestone
2023.01.01
Development

Successfully merging this pull request may close these issues.

Catalog issue when opening with a service other than the previous map
3 participants
@dsuren1 @MV88 @tdipisa