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(settings): Fix $userId dependency injection #48919

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

provokateurin
Copy link
Member

Summary

Fixes refactoring gone wrong in #48790.

Checklist

@provokateurin provokateurin added bug 3. to review Waiting for reviews labels Oct 28, 2024
@provokateurin provokateurin added this to the Nextcloud 31 milestone Oct 28, 2024
@provokateurin provokateurin changed the title fix(settings): Fix dependency injection fix(settings): Fix $userId dependency injection Oct 28, 2024
Copy link
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

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

Tested and works 👍

@provokateurin provokateurin merged commit e8dbf02 into master Oct 28, 2024
177 checks passed
@provokateurin provokateurin deleted the fix/settings/userid-dependency-injection branch October 28, 2024 10:44
@skjnldsv skjnldsv mentioned this pull request Jan 7, 2025
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.

3 participants