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

[stable27] fix: Avoid failing to update the current version entry if there is none #38970

Merged
merged 4 commits into from Jun 27, 2023
Merged

[stable27] fix: Avoid failing to update the current version entry if there is none #38970

merged 4 commits into from Jun 27, 2023

Conversation

backportbot-nextcloud[bot]
Copy link

Backport of #38800

@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 27.0.1 milestone Jun 23, 2023
@juliushaertl juliushaertl added the 4. to release Ready to be released and/or waiting for tests to finish label Jun 26, 2023
juliushaertl and others added 4 commits June 26, 2023 17:57
@juliushaertl @artonge
fix: Avoid failing to update the current version entry if there is none
ea30d11 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@artonge
Handle previousNode size === 0 on version creation
e8a2847 
Signed-off-by: Louis Chemineau <louis@chmn.me>
@juliushaertl @artonge
fix: Add logging statement for cases when updating the version fails
9d34895 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@artonge
Do not create a new version if previousNode has the same mtime
af0f114 
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge
Copy link
Contributor

artonge commented Jun 27, 2023

node workflow is not starting, but not needed in our case. Merging.

@artonge artonge merged commit ff04de6 into stable27 Jun 27, 2023
36 checks passed
@artonge artonge deleted the backport/38800/stable27 branch June 27, 2023 07:36
@blizzz blizzz mentioned this pull request Jul 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@artonge artonge artonge approved these changes

@come-nc come-nc come-nc approved these changes

@juliushaertl juliushaertl Awaiting requested review from juliushaertl

Assignees
No one assigned
Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Milestone
Nextcloud 27.0.1
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@artonge @come-nc @juliushaertl