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

fix: Also filter out undefined entries from the file list #42021

Merged
merged 2 commits into from
Dec 5, 2023

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl 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

@juliushaertl
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

@Fenn-CS Fenn-CS 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.

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

/compile

@juliushaertl
Copy link
Member Author

Manually backported both in #42023

@juliushaertl juliushaertl force-pushed the bugfix/noid/fix-undefined-array-entries branch from d88f1e4 to 7dc5cdc Compare December 5, 2023 10:09
@juliushaertl
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 /

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl juliushaertl force-pushed the bugfix/noid/fix-undefined-array-entries branch from 6ed4ea2 to 8b5eeb1 Compare December 5, 2023 15:57
@juliushaertl juliushaertl merged commit 3bdabae into master Dec 5, 2023
39 of 41 checks passed
@juliushaertl juliushaertl 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.

6 participants