Skip to content

Conversation

@JanisPlayer
Copy link
Contributor

@JanisPlayer JanisPlayer commented May 3, 2023

☑️ Resolves

I add 'preview_imaginary_url', 'preview_imaginary_key','preview_format', 'webp_quality'.

  • Fix And I updated jpeg quality from 90 to 80 as it actually says in the code.

🖼️ Screenshots

image

@artonge
Copy link
Contributor

artonge commented May 3, 2023

Let's wait until nextcloud/server#38032 is merged, but this is perfect ❤️

@JanisPlayer
Copy link
Contributor Author

I have already adjusted the doc a bit.
preview_imaginary_key
For the other previews options you could rename preview_format to preview_imaginary_format, because ImageMagick does not change anything.
Should there also be a mention of the key here?
Server Tuning

@szaimen
Copy link
Contributor

szaimen commented Sep 2, 2023

@JanisPlayer please adjust the docs now that nextcloud/server#38032 is merged :)

@JanisPlayer JanisPlayer changed the title Update previews_configuration.rst add 'preview_imaginary_url', 'preview_imaginary_key','preview_format', 'webp_quality' Sep 2, 2023
Signed-off-by: Simon L <[email protected]>

Signed-off-by: Simon L. <[email protected]>
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

Looks good

@szaimen szaimen enabled auto-merge September 5, 2023 09:20
@szaimen szaimen merged commit fe54597 into nextcloud:master Sep 5, 2023
@welcome
Copy link

welcome bot commented Sep 5, 2023

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/documentation/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

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.

3 participants