Skip to content

feat: add logging to preview generation - #52221

Merged
nfebe merged 1 commit into
masterfrom
feat/no-issue/add-logging-preview-generation
Apr 22, 2025
Merged

feat: add logging to preview generation#52221
nfebe merged 1 commit into
masterfrom
feat/no-issue/add-logging-preview-generation

Conversation

@nfebe

Copy link
Copy Markdown
Contributor

Improve logging and as a result debugging for preview generation issues

@nfebe
nfebeforce-pushed the feat/no-issue/add-logging-preview-generation branch from 98e5bcd to c31c8aeCompareApril 16, 2025 18:00
@nfebe
nfebe marked this pull request as ready for review April 16, 2025 18:04
@nfebe
nfebe requested a review from a team as a code ownerApril 16, 2025 18:04
@nfebe
nfebe requested review from ArtificialOwl, artonge, come-nc, skjnldsv and sorbaugh and removed request for a team and come-ncApril 16, 2025 18:04
@nfebenfebe added enhancement 3. to review Waiting for reviews labels Apr 16, 2025
@github-project-automationgithub-project-automationBot moved this to 🏗️ In progress in 📁 Files teamApr 16, 2025
@nfebe
nfebeforce-pushed the feat/no-issue/add-logging-preview-generation branch from c31c8ae to 2f5eff2CompareApril 16, 2025 18:56
@nfebe
nfebeforce-pushed the feat/no-issue/add-logging-preview-generation branch from 2f5eff2 to f185a58CompareApril 17, 2025 10:22
@nfebe
nfebe enabled auto-merge April 17, 2025 10:22
@nfebe
nfebeforce-pushed the feat/no-issue/add-logging-preview-generation branch from f185a58 to 03c3630CompareApril 17, 2025 11:50
Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe
nfebeforce-pushed the feat/no-issue/add-logging-preview-generation branch from 03c3630 to 6b89838CompareApril 22, 2025 09:41
@nfebe
nfebe merged commit 25bc18c into masterApr 22, 2025
@nfebe
nfebe deleted the feat/no-issue/add-logging-preview-generation branch April 22, 2025 10:37
@nfebe

Copy link
Copy Markdown
ContributorAuthor

/backport to stable31

AndyXheli added a commit to AndyXheli/server that referenced this pull request May 15, 2025
Since this PR nextcloud#52221 was implemented, the log file has been flooded with warnings stating, "Cached preview not found for file; generating a new preview." This appears to be more of an informational message rather than a warning. This PR will change it from warning to debug
Original PR
nextcloud#52221
Signed-off-by: AndyXheli <andyxheli@gmail.com>
backportbotBot pushed a commit that referenced this pull request May 16, 2025
Since this PR #52221 was implemented, the log file has been flooded with warnings stating, "Cached preview not found for file; generating a new preview." This appears to be more of an informational message rather than a warning. This PR will change it from warning to debug
Original PR
#52221
Signed-off-by: AndyXheli <andyxheli@gmail.com>
AIlkiv pushed a commit to AIlkiv/server that referenced this pull request Jul 10, 2025
Since this PR nextcloud#52221 was implemented, the log file has been flooded with warnings stating, "Cached preview not found for file; generating a new preview." This appears to be more of an informational message rather than a warning. This PR will change it from warning to debug
Original PR
nextcloud#52221
Signed-off-by: AndyXheli <andyxheli@gmail.com>
@nextcloud-botnextcloud-bot mentioned this pull request Aug 19, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancement

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

3 participants

@nfebe@artonge@skjnldsv