Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Dec 4, 2023

Apparently I misread the cypress failure in #42006 which was actually related. This pr fixes that to also filter out undefined values from the list e.g. when a file was moved to another folder

@juliusknorr juliusknorr added bug 3. to review Waiting for reviews labels Dec 4, 2023
@juliusknorr juliusknorr requested review from a team, AndyScherzinger, ChristophWurst, emoral435, nfebe and sorbaugh and removed request for a team December 4, 2023 21:50
@juliusknorr juliusknorr added this to the Nextcloud 29 milestone Dec 4, 2023
@juliusknorr
Copy link
Member Author

/backport to stable28

@AndyScherzinger
Copy link
Member

👍

Also the backport of the fix hasn't been picked up by the bot ye6t

Copy link
Contributor

@nfebe nfebe left a comment

Choose a reason for hiding this comment

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

Cypress green.

@juliusknorr juliusknorr force-pushed the bugfix/noid/fix-undefined-array-entries branch from f0f7840 to 1583484 Compare December 5, 2023 07:36
@juliusknorr
Copy link
Member Author

/compile

@juliusknorr
Copy link
Member Author

Manually backported both in #42023

@juliusknorr juliusknorr force-pushed the bugfix/noid/fix-undefined-array-entries branch from d88f1e4 to 7dc5cdc Compare December 5, 2023 10:09
@juliusknorr
Copy link
Member Author

/compile

@AndyScherzinger AndyScherzinger force-pushed the bugfix/noid/fix-undefined-array-entries branch from 7dc5cdc to 6ed4ea2 Compare December 5, 2023 11:01
@AndyScherzinger
Copy link
Member

/compile amend /

@juliusknorr juliusknorr force-pushed the bugfix/noid/fix-undefined-array-entries branch from 6ed4ea2 to 8b5eeb1 Compare December 5, 2023 15:57
@juliusknorr juliusknorr merged commit 3bdabae into master Dec 5, 2023
@juliusknorr juliusknorr deleted the bugfix/noid/fix-undefined-array-entries branch December 5, 2023 19:52
@blizzz blizzz mentioned this pull request Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants