-
-
Notifications
You must be signed in to change notification settings - Fork 326
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
[14.0][FIX]attachment_preview: get preview for files in db only #485
[14.0][FIX]attachment_preview: get preview for files in db only #485
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I tested it locally, and it worked as expected.
@PicchiSeba, if possible, please rebase so that is available the runboat so that others can review it.
244c967
to
06a0843
Compare
Hi @kaynnan , thank you for your review. I just rebased :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR has the |
/ocabot merge patch |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at b315d47. Thanks a lot for contributing to OCA. ❤️ |
In a few cases, when a user uploads files such as PDFs, when the client tries to get the preview, the file may not be saved yet.
This results in the following error
![image001](https://private-user-images.githubusercontent.com/38473030/330089458-30d97467-3e42-49d9-8354-ce5407be37e9.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjEyNjM4NzEsIm5iZiI6MTcyMTI2MzU3MSwicGF0aCI6Ii8zODQ3MzAzMC8zMzAwODk0NTgtMzBkOTc0NjctM2U0Mi00OWQ5LTgzNTQtY2U1NDA3YmUzN2U5LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzE4VDAwNDYxMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTMwMzE3MjRhYjVjNjFiMDE3ZjE4YTMyZWQ1ZGYyZjQ3MTc5NTc4YzI3NmYxOGU5ODE3NDU5ZWZmZmJkZGU4MDUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.fXNOjSrl4hftSotiKfOEF9VmsmqveEPzlQ1ccarJBcY)
With this check we fetch only the previews of those files we know the id of.