Skip to content

Conversation

@Pytal
Copy link
Member

@Pytal Pytal commented Aug 21, 2024

Summary

Technically this is a feature but could also be considered a fix. Favorites were being loaded with a hardcoded limit

public const FAVORITE_LIMIT = 50;
of 50, so once a user had >=50 favorites none would be loaded. To fix this we no longer block page load with the server-side retrieval and load them from the frontend instead.

Checklist

@Pytal Pytal added this to the Nextcloud 31 milestone Aug 21, 2024
@Pytal Pytal requested a review from a team August 21, 2024 17:37
@Pytal Pytal self-assigned this Aug 21, 2024
@Pytal Pytal requested a review from skjnldsv as a code owner August 21, 2024 17:37
@Pytal Pytal requested review from artonge and nfebe and removed request for a team August 21, 2024 17:37
@Pytal Pytal enabled auto-merge August 21, 2024 17:39
@Pytal

This comment was marked as resolved.

Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

Nice! Fix, cleanup and semi-feature 😄

@susnux susnux force-pushed the feat/load-more-than-50-faves branch 2 times, most recently from 4f911c4 to c887c14 Compare August 24, 2024 10:17
@susnux
Copy link
Contributor

susnux commented Aug 24, 2024

/backport to stable30

@susnux susnux force-pushed the feat/load-more-than-50-faves branch from c887c14 to 36c6fc6 Compare August 24, 2024 10:37
@Pytal Pytal force-pushed the feat/load-more-than-50-faves branch from 36c6fc6 to 7117356 Compare August 26, 2024 18:05
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

Pytal and others added 3 commits August 27, 2024 14:33
@Pytal Pytal force-pushed the feat/load-more-than-50-faves branch from 7117356 to a1f4396 Compare August 27, 2024 22:39
@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 27, 2024
@Pytal
Copy link
Member Author

Pytal commented Aug 27, 2024

/compile /

Signed-off-by: nextcloud-command <[email protected]>
@Pytal Pytal merged commit b7212e3 into master Aug 27, 2024
@Pytal Pytal deleted the feat/load-more-than-50-faves branch August 27, 2024 23:25
@skjnldsv skjnldsv mentioned this pull request Jan 7, 2025
@susnux
Copy link
Contributor

susnux commented Jun 23, 2025

For 29: #53643

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug enhancement feature: files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Favorites in Files navigation not expandable anymore

6 participants