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

enh: add some more documentation for antivirus usage #10711

Merged
merged 2 commits into from
Jan 7, 2024

Conversation

max-nextcloud
Copy link
Contributor

@max-nextcloud max-nextcloud commented Jul 4, 2023

grafik

@max-nextcloud
Copy link
Contributor Author

nextcloud/files_antivirus#284 added the commands.

@kesselb
Copy link
Contributor

kesselb commented Aug 3, 2023

Hi, thanks for your nice pull request 👍

We have a different page for most of the occ commands: https://github.com/nextcloud/documentation/blob/master/admin_manual/configuration_server/occ_command.rst

Do you think it makes sense to move them over there?

@blizzz
Copy link
Member

blizzz commented Nov 23, 2023

Please don't forget to set the milestone when opening a PR 😃

@max-nextcloud
Copy link
Contributor Author

I tried to include all feedback:

  • typos fixed
  • added occ commands also on the occ command page

I also changed the occ commands in the antivirus docs to include sudo -u www-data php to match the ones in the occ docs.

@luka-nextcloud @kesselb could you have another look?

@max-nextcloud max-nextcloud merged commit f208af5 into master Jan 7, 2024
11 checks passed
@max-nextcloud max-nextcloud deleted the enh/virus-scanner-docs branch January 7, 2024 21:46
@max-nextcloud
Copy link
Contributor Author

/backport stable28

acataluddi pushed a commit to acataluddi/nextcloud-documentation that referenced this pull request Apr 28, 2024
Including feedback from PR review of nextcloud#10711.

Signed-off-by: Max <[email protected]>
Signed-off-by: Adriano Cataluddi <[email protected]>
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.

6 participants