SUAI-TaskPlanner-Contest / TaskPlanner

Client application for working with todos and syncing with CalDAV servers
MIT License
3 stars 2 forks source link

31 nextcloud instruction #32

Closed pavelparfishov closed 1 year ago

pavelparfishov commented 1 year ago

Task merge request

Related Issue | [Создать инструкцию по установке Nextcloud #31](https://github.com/SUAI-TaskPlanner-Contest/TaskPlanner/issues/31) -- | -- Квалификация ревьюера | Знание требований заказчика, навыки работы с UNIX системами Исполнители: @pavelparfishov

Code changes

Была составлена инструкция для развертывания Nextcloud на Ubuntu 22.04

UI Demonstration

![image](https://user-images.githubusercontent.com/78814540/230663305-2f5ee3ed-1776-4c32-a5d8-95cfb929c832.png)

How to test

Прочитать инструкцию

astronik00 commented 1 year ago

@pavelparfishov

  1. Ты указываешь, что ставишь версию Ubuntu 23, image

но далее в тексте приводишь команду для 22.10 версии. image

Я могу конечно догадаться, что раз у тебя все работает, то и команда рабочая и для 23 версии, но поправь пожалуйста версии.

  1. Код записывается следующим образом:

    // какой-то код

    Во-первых, плюс в том, что код более контрастный на фоне остального текста, а, во-вторых, справа есть удобная кнопка для копирования кода

  1. Текст не соответствует командам

Следующая команда позволяет всего лишь подключиться к MySQL с правами суперпользователя через sudo:

sudo mysql -u root -p

Также насколько я знаю, ключ -p не обязателен здесь, поскольку при создании БД пароль root изначально пуст

Здесь мне не нравится описание следующей команды:

CREATE DATABASE nextcloud;

CREATE USER 'nextclouduser'@'localhost' IDENTIFIED BY 'password';

GRANT ALL PRIVILEGES ON nextcloud.* TO 'nextclouduser'@'localhost';

FLUSH PRIVILEGES;

EXIT;

Я хочу, чтобы было понятно, что эта команда делает: Эта команда позволяет создать пользователя с заданным именем и паролем, а затем передать ему все права для работы с только что созданной базой данных <имя_бд>

Добавь сюда команду, которая теперь будет использоваться, чтобы подключиться за только что созданного пользователя

  1. По пункту "Установите NextCloud"

Не надо устраивать месиво из команд. Раздели пожалуйста их по отдельности и опиши каждую, что она примерно делает, а также укажи, что это все делается из терминала Ubuntu (это перед всеми командами нужно указать, еще до mySQL):

Скачать nextcloud:

wget https://download.nextcloud.com/server/releases/latest.tar.bz2

Разархивировать скачанный пакет:

sudo tar -xjf latest.tar.bz2 -C /var/www/html/

Зачем-то менять владельца? В общем сюда умный текст:

sudo chown -R www-data:www-data /var/www/html/nextcloud/
  1. Настройки Apache Оставь только готовый кусок настроек для localhost. Кстати, подозреваю, что необходимо использовать команду Dav on, ибо в твоем примере она Dav off, а это, вероятно всего, и есть включение CalDAV. Необходимо разобраться
pavelparfishov commented 1 year ago

@astronik00

  1. Исправил версии
  2. Исправил форматирование кода
  3. Добавил/скорректировал описание команд
  4. По поводу твоего пункта 5 (Настройки Apache):

Dav off в данном контексте отключает модуль WebDAV (Web-based Distributed Authoring and Versioning) для данного виртуального хоста в Apache.

WebDAV - это расширение протокола HTTP, которое позволяет работать с файлами на сервере через интернет, как если бы они были на локальном компьютере. Например, пользователи могут загружать файлы, переименовывать их, удалять их и т.д. через веб-браузер, используя специальные программы, которые поддерживают WebDAV.

В данном конфигурационном файле для Apache, Dav off указывает, что модуль WebDAV должен быть отключен для данного виртуального хоста. Это может быть полезно для повышения безопасности сервера, если WebDAV не используется на сайте.

WebDAV и CalDAV - это два разных протокола, но CalDAV является расширением WebDAV, которое позволяет работать с календарями и задачами на сервере через интернет.

CalDAV использует WebDAV для работы с календарями и задачами, но при этом не требует, чтобы сам WebDAV был включен на сервере. Это означает, что если на сервере включен только CalDAV, то WebDAV не нужен для работы CalDAV.

Нужно понять, нужен ли нам вообще WebDAV 🤔

astronik00 commented 1 year ago

@pavelparfishov Если WebDAV работает в таком случае, как синхронизация самих файлов сервера, то не нужен. Тогда предположу, что CalDAV идет уже встроен в конкретное приложение календаря

astronik00 commented 1 year ago

@pavelparfishov Также при установке этой версии у меня вылезла такая ошибка: image

Я решила это следующим образом: При выключенной машине зайти в:

  1. Настроить -> Дисплей -> Видеопамять, поднять до 128 мб
  2. Настроить -> Система -> Процессоры, поднять до 4

Не знаю, что решило точно, но только после этого началась установка. Думаю, это важно добавить, чтобы избежать такой ошибки

pavelparfishov commented 1 year ago

@pavelparfishov Также при установке этой версии у меня вылезла такая ошибка: image

Я решила это следующим образом: При выключенной машине зайти в:

  1. Настроить -> Дисплей -> Видеопамять, поднять до 128 мб
  2. Настроить -> Система -> Процессоры, поднять до 4

Не знаю, что решило точно, но только после этого началась установка. Думаю, это важно добавить, чтобы избежать такой ошибки

Добавил пункт - 1.6 Рекомендации по настройке виртуальной машины

astronik00 commented 1 year ago

@pavelparfishov К сожалению, дальше проверять не могу, у меня убунта каждый раз умирает на распаковке архива. Тут уже проблемы производительности моего ноута. Попрошу посмотреть кого-то с помощнее ПК

pavelparfishov commented 1 year ago

@pavelparfishov К сожалению, дальше проверять не могу, у меня убунта каждый раз умирает на распаковке архива. Тут уже проблемы производительности моего ноута. Попрошу посмотреть кого-то с помощнее ПК

Хорошо

astronik00 commented 1 year ago

@pavelparfishov Пункты 1.2 и 1.6 лучше объединить вместе. Можно сделать так, сперва рекомендации разработчиков, а дальше проблемы работяг

astronik00 commented 1 year ago

@pavelparfishov Снова у тебя 2 разные версии Ubuntu фигурируют. Делаешь для 23, а ссылку на скачивание оставил на 22.04. Верни ссылку на 23.

В пункте 2.3 подпункт 3 масло масляное, читать невозможно. Убери второе предложение со звездочкой

Пункт 2.4 последняя команда - тоже. Что за копипаст? Текст ради текста, хотя смысл команды можно было изложить в одном предложении: "Измените владельца и группу в директории /var/www/html/nextcloud/ на пользователя www-data и группу www-data, чтобы обеспечить корректную работу nextcloud".

Если не будешь редактировать нормально то, что тебе дает этот аналог chatGPT, я не буду принимать работу с ее использованием.

astronik00 commented 1 year ago

@pavelparfishov Исправь мои последние замечания и я закрою MR.

pavelparfishov commented 1 year ago

@astronik00

@pavelparfishov Снова у тебя 2 разные версии Ubuntu фигурируют. Делаешь для 23, а ссылку на скачивание оставил на 22.04. Верни ссылку на 23.

Ссылку на 23 я не убирал (пункт 2), просто добавил ссылку на рекомендованную версию (пункт 1). Раз не нужна - удалил.

В пункте 2.3 подпункт 3 масло масляное, читать невозможно. Убери второе предложение со звездочкой

Удалил.

Пункт 2.4 последняя команда - тоже. Что за копипаст? Текст ради текста, хотя смысл команды можно было изложить в одном предложении: "Измените владельца и группу в директории /var/www/html/nextcloud/ на пользователя www-data и группу www-data, чтобы обеспечить корректную работу nextcloud".

Добавил твой вариант.