Fix thumbnail extension to png #180
Conversation
…ually a png This fixes thumbnail uploading for AYON server 1.5.5
There was a problem hiding this comment.
Inspected thumbnail representation details of the review and it correctly uses PNG as seen on the details pane below:
However not sure if that should also be used for thumbnail captured via Publisher UI as my model have it and it says JPG been used as repre type for model product thumbnail
If its not related to this PR, just tell me and I will give approval asap...thx
Where do you see it mentioned to be |
Its in the representation details on the thumbnail repre for the model product....that one comes from Publisher UI and was added by me before publishing... still JPG (but maybe it really is:) thats the reason I was asking if valid or not... 
|
| @@ -74,7 +74,7 @@ def process(self, instance): | |||
|
|
|||
| representation = { | |||
| "name": "thumbnail", | |||
| "ext": "jpg", | |||
| "ext": "png", | |||
| "files": thumbnail, | |||
There was a problem hiding this comment.
Maybe the thumbnail variable holds .jpeg?
There was a problem hiding this comment.
That's a very good point!
Changelog Description
Change representation extension to
pngbecause the thumbnail is actually a pngAdditional review information
This fixes thumbnail uploading for AYON server 1.5.5
Testing notes: