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

Allow specify a config prefix for another database connection #30973

Merged
merged 1 commit into from
Feb 7, 2022
Merged

Allow specify a config prefix for another database connection #30973

merged 1 commit into from
Feb 7, 2022

Conversation

nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Feb 2, 2022 •
edited
Loading

@nickvergessen nickvergessen added this to the Nextcloud 24 milestone Feb 2, 2022
@nickvergessen nickvergessen added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Feb 2, 2022
This was referenced Feb 2, 2022
Merged
Closed
@nickvergessen
Allow specify a config prefix for another database connection
7654117 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the techdebt/noid/allow-spliting-activities-to-another-database branch from fb0b1ef to 7654117 Compare February 3, 2022 16:16
@nickvergessen nickvergessen added 3. to review Waiting for reviews performance 🚀 and removed 2. developing Work in progress labels Feb 3, 2022
@nickvergessen nickvergessen marked this pull request as ready for review February 3, 2022 16:17
PVince81
PVince81 approved these changes Feb 3, 2022
View reviewed changes
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@PVince81 PVince81 requested a review from artonge February 7, 2022 10:58
@PVince81 PVince81 merged commit a7925a6 into master Feb 7, 2022
@PVince81 PVince81 deleted the techdebt/noid/allow-spliting-activities-to-another-database branch February 7, 2022 14:16
@PVince81
Copy link
Member

PVince81 commented Feb 7, 2022

/backport to stable23

@PVince81
Copy link
Member

PVince81 commented Feb 7, 2022

/backport to stable22

@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request Feb 7, 2022
Merged
@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request Feb 7, 2022
Merged
@nickvergessen nickvergessen mentioned this pull request Feb 22, 2022
Closed
This was referenced Feb 22, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PVince81 PVince81 PVince81 approved these changes

@artonge artonge artonge approved these changes

@icewind1991 icewind1991 Awaiting requested review from icewind1991

@juliusknorr juliusknorr Awaiting requested review from juliusknorr

@CarlSchwan CarlSchwan Awaiting requested review from CarlSchwan

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

Assignees
No one assigned
Labels
3. to review Waiting for reviews performance 🚀 technical debt
Projects
None yet
Milestone
Nextcloud 24
Development

Successfully merging this pull request may close these issues.
4 participants
@nickvergessen @PVince81 @artonge @jes-github