Skip to content
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

Doc tomdereub #737

Closed
wants to merge 4 commits into from
Closed

Conversation

tomdereub
Copy link
Contributor

@tomdereub tomdereub commented Oct 4, 2024

Problem/Solution

Add doc about file retention, linked to #612.

PR Status

  • Code finished and ready to be reviewed

Add doc about file retention.
Sortir backend hautes performances du chapitre onlyoffice.
Doc about file retention in french.
@MCMic
Copy link

MCMic commented Oct 22, 2024

The link to Nextcloud documentation should be to latest, not 18. Or to the packaged version in this repo, so hardcoded 29 or 30.

@ericgaspar
Copy link
Member

I feel that, even if the information is relevant in itself, we should not clutter our doc by duplicating that of Nextcloud.

@tomdereub
Copy link
Contributor Author

I would see 2 ways of looking at it :

  • yunohost package ships default nextcloud config, as for now, and doesn't "duplicate" nextcloud's doc.
  • yunohost package includes some specific config, as demanded in Add trashbin cleanup and file versioning #612, and the package doc must explain this

Yunohost goal is to ease server admin for people that have limited skills in server administration, so in my opinion it's better to add a specific config that will avoid problems for most users (full storage in that case), and document it.
But that's just my humble opinion...

@ericgaspar
Copy link
Member

If we want to ease server admin, I would rather include the option in config panel like proposed in this PR #638

@ericgaspar
Copy link
Member

closing in favor of #638

@ericgaspar ericgaspar closed this Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants