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

issue 16039: update versionCount also if versions are disabled #16120

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

chm0815
Copy link
Contributor

@chm0815 chm0815 commented Oct 18, 2023

Changes in this pull request

Resolves #issue 16039

HOW

The versionCount will now also be set when version creation is disabled for the the current process.

@github-actions
Copy link

Review Checklist

  • Target branch (11.0 for bug fixes, others 11.x)
  • Tests (if it's testable code, there should be a test for it - get help)
  • Docs (every functionality needs to be documented, see here)
  • Migration incl. install.sql (e.g. if the database schema changes, ...)
  • Upgrade notes (deprecations, important information, migration hints, ...)
  • Label
  • Milestone

@sonarcloud
Copy link

sonarcloud bot commented Oct 18, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@dvesh3
Copy link
Contributor

dvesh3 commented Oct 30, 2023

Hi @chm0815 Pimcore 10.6 reached EOL so we don't accept contributions for this version anymore. Please rebase it on branch 11.1 only if it is reproducible on latest version v11.1.0 otherwise please close this PR. thanks for your understanding 😊

@chm0815 chm0815 changed the base branch from 10.6 to 11.1 October 30, 2023 16:03
@chm0815
Copy link
Contributor Author

chm0815 commented Oct 30, 2023

Rebased the PR to pimcore 11.1

@dvesh3
Copy link
Contributor

dvesh3 commented Nov 7, 2023

the failing test is not related to the changes here.

@dvesh3 dvesh3 added this to the 11.1.1 milestone Nov 7, 2023
@dvesh3 dvesh3 merged commit 7ef4ae5 into pimcore:11.1 Nov 7, 2023
12 of 14 checks passed
@dvesh3
Copy link
Contributor

dvesh3 commented Nov 7, 2023

@chm0815 thanks!

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

Successfully merging this pull request may close these issues.

None yet

3 participants