Skip to content

Bump files app version #40878

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

Merged
merged 3 commits into from
Jul 17, 2023
Merged

Bump files app version #40878

merged 3 commits into from
Jul 17, 2023

Conversation

pako81
Copy link

@pako81 pako81 commented Jul 17, 2023

Description

Bump files app version for adding missing background jobs.

Related Issue

Motivation and Context

Files app version was not properly increased when adding the OCA\Files\BackgroundJob\CleanupPersistentFileLock and OCA\Files\BackgroundJob\PreviewCleanupJob jobs. As a result, those jobs were not correctly inserted into the oc_jobs table upon a core upgrade. First time installations are not affected as there jobs are correctly added.

How Has This Been Tested?

  • Manually increasing the version to 1.6.0 in the app info.xml file and observing the two jobs are inserted into the oc_jobs table after running occ upgrade.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised
  • Changelog item

@pako81 pako81 added this to the development milestone Jul 17, 2023
@pako81 pako81 requested review from jvillafanez and phil-davis July 17, 2023 11:24
@pako81 pako81 self-assigned this Jul 17, 2023
@owncloud owncloud deleted a comment from update-docs bot Jul 17, 2023
@sonarqubecloud
Copy link

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

@pako81 pako81 merged commit 597d849 into master Jul 17, 2023
@delete-merged-branch delete-merged-branch bot deleted the bump_files_app_version branch July 17, 2023 13:16
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.

2 participants