Skip to content

Conversation

CarlSchwan
Copy link
Member

@CarlSchwan CarlSchwan commented Aug 13, 2025

Summary

Required for nextcloud/deck#7164

Checklist

@CarlSchwan CarlSchwan self-assigned this Aug 13, 2025
@CarlSchwan CarlSchwan force-pushed the comments-db-multiple branch from 4b08704 to 9a7a0c2 Compare August 13, 2025 13:03
@CarlSchwan CarlSchwan marked this pull request as ready for review August 13, 2025 14:06
@CarlSchwan CarlSchwan requested review from a team and nickvergessen as code owners August 13, 2025 14:06
@CarlSchwan CarlSchwan requested review from Altahrim, nfebe and salmart-dev and removed request for a team August 13, 2025 14:06
@CarlSchwan CarlSchwan force-pushed the comments-db-multiple branch 2 times, most recently from c623447 to 17135e2 Compare August 13, 2025 15:37
Copy link
Member

@icewind1991 icewind1991 left a comment

Choose a reason for hiding this comment

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

Good otherwise

@CarlSchwan CarlSchwan force-pushed the comments-db-multiple branch 2 times, most recently from 48fa3fb to 07fac3b Compare August 14, 2025 11:18
@CarlSchwan CarlSchwan force-pushed the comments-db-multiple branch from 07fac3b to f91fe6a Compare August 14, 2025 17:10
@salmart-dev
Copy link
Contributor

@CarlSchwan master has some changes from the preloading event branch, when you rebase you will need to do some adaptations. Let me know if you need help and when you're done ping me for a review!

… same time

Signed-off-by: Carl Schwan <carl.schwan@nextclound.com>
@CarlSchwan CarlSchwan force-pushed the comments-db-multiple branch 2 times, most recently from cd3ac4a to daef65a Compare August 21, 2025 09:30
@CarlSchwan
Copy link
Member Author

@CarlSchwan master has some changes from the preloading event branch, when you rebase you will need to do some adaptations. Let me know if you need help and when you're done ping me for a review!

rebased :)

Since we now have an easy way to fetch the comments count.

Signed-off-by: Carl Schwan <carl.schwan@nextclound.com>
@CarlSchwan CarlSchwan force-pushed the comments-db-multiple branch from daef65a to 94e2af0 Compare August 21, 2025 09:31
@CarlSchwan CarlSchwan added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 25, 2025
@CarlSchwan CarlSchwan merged commit d0f819b into master Aug 27, 2025
246 of 254 checks passed
@CarlSchwan CarlSchwan deleted the comments-db-multiple branch August 27, 2025 09:57
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 28, 2025
Sign up for free to 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.

5 participants