Skip to content

Delay loading user preferences until we need them - #31363

Merged
nickvergessen merged 1 commit into
masterfrom
techdebt/noid/delay-getting-preferences
Mar 1, 2022
Merged

Delay loading user preferences until we need them#31363
nickvergessen merged 1 commit into
masterfrom
techdebt/noid/delay-getting-preferences

Conversation

@nickvergessen

@nickvergessennickvergessen commented Feb 25, 2022

Copy link
Copy Markdown
Member

Helps in certain requests to drastically reduce the number of queries

Bildschirmfoto von 2022-02-26 00-33-23

@nickvergessennickvergessen added this to the Nextcloud 24 milestone Feb 25, 2022
@nickvergessen
nickvergessenforce-pushed the techdebt/noid/delay-getting-preferences branch from bff1326 to 515808fCompareFebruary 28, 2022 12:40
@nickvergessennickvergessen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Feb 28, 2022
@nickvergessen
nickvergessen marked this pull request as ready for review February 28, 2022 12:40
@nickvergessen
nickvergessen requested review from a team, CarlSchwan, PVince81, blizzz, danxuliu and vitormattos and removed request for a team and blizzzFebruary 28, 2022 12:41

@CarlSchwanCarlSchwan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, just one small thing to fix

Comment threadlib/private/User/User.php Outdated
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the techdebt/noid/delay-getting-preferences branch from 515808f to b90e657CompareFebruary 28, 2022 12:49

@PVince81PVince81 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

good idea 👍

@nickvergessen
nickvergessen merged commit 46e6c49 into masterMar 1, 2022
@nickvergessen
nickvergessen deleted the techdebt/noid/delay-getting-preferences branch March 1, 2022 09:06
Sign up for freeto 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.

3 participants

@nickvergessen@PVince81@CarlSchwan