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(TaskProcessingApiController): Don't use + to merge non-assoc. arrays #46222

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

marcelklehr
Copy link
Member

Summary

I used + when I should have used array_merge 🙈

Checklist

@marcelklehr marcelklehr requested review from julien-nc and bigcat88 July 1, 2024 13:05
@marcelklehr marcelklehr force-pushed the fix/task-processing-api-controller/dont-use-plus branch from 2801b24 to 0574d7e Compare July 1, 2024 14:29
@solracsf solracsf added the 3. to review Waiting for reviews label Jul 1, 2024
@solracsf solracsf added this to the Nextcloud 30 milestone Jul 1, 2024
@marcelklehr marcelklehr force-pushed the fix/task-processing-api-controller/dont-use-plus branch from 0574d7e to 224779c Compare July 1, 2024 16:47
This was referenced Jul 30, 2024
This was referenced Aug 5, 2024
@skjnldsv skjnldsv mentioned this pull request Aug 13, 2024
@skjnldsv skjnldsv modified the milestones: Nextcloud 30, Nextcloud 31 Aug 14, 2024
@skjnldsv skjnldsv merged commit 452e4be into master Nov 6, 2024
166 checks passed
@skjnldsv skjnldsv deleted the fix/task-processing-api-controller/dont-use-plus branch November 6, 2024 08:02
@marcelklehr
Copy link
Member Author

marcelklehr commented Nov 6, 2024

/backport to stable30

1 similar comment
@marcelklehr
Copy link
Member Author

/backport to stable30

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants