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

🐛 Remove the concatenation from query builder and #7312

Merged
merged 1 commit into from
Oct 13, 2022

Conversation

miaulalala
Copy link
Contributor

replace with a comma instead...

Copy link
Member

@CarlSchwan CarlSchwan left a comment

Choose a reason for hiding this comment

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

🥇

@miaulalala
Copy link
Contributor Author

Can't merge bc integration tests are not afjusted yet.

@ChristophWurst
Copy link
Member

Can't merge bc integration tests are not afjusted yet.

Could you adjust them please?

@miaulalala
Copy link
Contributor Author

Can't merge bc integration tests are not afjusted yet.

Could you adjust them please?

#7306

@miaulalala miaulalala force-pushed the fix/semantic-error-in-unanalyzed-method branch from 2bfbc0a to 785ef11 Compare September 29, 2022 14:05
@miaulalala
Copy link
Contributor Author

/rebase

@ChristophWurst
Copy link
Member

/backport to stable2.0

replace with a comma instead...

Signed-off-by: Anna Larch <anna@nextcloud.com>
@miaulalala miaulalala force-pushed the fix/semantic-error-in-unanalyzed-method branch from 785ef11 to ec28585 Compare October 13, 2022 10:12
@nextcloud nextcloud deleted a comment from ChristophWurst Oct 13, 2022
@GretaD
Copy link
Contributor

GretaD commented Oct 13, 2022

/backport to stable1.14

@miaulalala miaulalala merged commit 6112cc5 into main Oct 13, 2022
@miaulalala miaulalala deleted the fix/semantic-error-in-unanalyzed-method branch October 13, 2022 14:59
@backportbot-nextcloud
Copy link

The backport to stable1.14 failed. Please do this backport manually.

@ChristophWurst
Copy link
Member

2dcf8b9 was only added for v2.0

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.

5 participants