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

Merge release/15.0 into dev #17251

Merged
merged 20 commits into from
Nov 22, 2024
Merged

Merge release/15.0 into dev #17251

merged 20 commits into from
Nov 22, 2024

Conversation

openprojectci
Copy link
Contributor

Created by GitHub action

jjabari-op and others added 20 commits November 18, 2024 17:54
- https://community.openproject.org/wp/59391
- files tab counter now only calls for file links of wp with page size 0
- file links endpoint returns paginated collections
- frontend requests file links with page size -1 (INFINITE)
- file sync is still executed for ALL file links of a work package, not
  only the requested page
  - this is practically no change for the product, as we do not fetch
    single pages
  - but fetching only the total numbers with page size 0 now does not
    trigger a sync
- avoid sql injection
- file link collection is returned ordered by id asc
…he-new-activity-tab-renders-quickly-also-for-work-packages-with-over-100-comments
…he-new-activity-tab-renders-quickly-also-for-work-packages-with-over-100-comments
Co-authored-by: Kabiru Mwenja <k.mwenja@openproject.com>
…g-in-files-tab-wp-full-view

[#59391] rework file links to paginated collection
…-the-new-activity-tab-renders-quickly-also-for-work-packages-with-over-100-comments

Code maintenance/59280 ensure that the new activity tab renders quickly also for work packages with over 100 comments
@openprojectci openprojectci added the create-merge-release-into-dev-pr Mark auto created PRs that will be auto closed too label Nov 22, 2024
@ulferts ulferts merged commit 91509bb into dev Nov 22, 2024
@ulferts ulferts deleted the merge-release/15.0-20241122034127 branch November 22, 2024 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
create-merge-release-into-dev-pr Mark auto created PRs that will be auto closed too
Development

Successfully merging this pull request may close these issues.

6 participants