SUAI-TaskPlanner-Contest / TaskPlanner

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

18 create a transition diagram #33

Closed Dary5 closed 1 year ago

Dary5 commented 1 year ago

Task merge request

Related Issue #18
Reviewer requrements Умение разбираться в диаграмме

UI Demonstration

image

How to test

Необходимо убедиться, что весь функционал для Пользователя, описанные в ТЗ, учтён.

astronik00 commented 1 year ago

@Dary5 Смотри пример оформления MR или здесь.

astronik00 commented 1 year ago

@Dary5 Даша, на Issue нужно в таблице указывать ссылку, чтобы любой человек мог легко увидеть, над чем ты работала

Давай еще добавим возможность делать поиск по кнопке по задачам

aleksandra-shchegoleva commented 1 year ago

@Dary5 @astronik00 1) Кнопка "Синхронизация с сервером" - будет ли какое-то всплывающее сообщение по выбору приоритета? Человек может нажать на эту кнопку и не знать какой приоритет сейчас стоит (особенно, если он первый раз пользуется). Или же можно вынести кнопку выбора приоритета на главное окно (например, рядом с комбобоксом, где сервер выбирается) 2) Опечатка image 3) ТЗ-функциональные требования-пункт 7 "Реализовать возможность сохранения логина и пароля пользователя в настройках." Не совсем поняла, где на диаграмме этот момент. Пароль и логин сохранятеся при нажатии кнопки "Подключить" или это поля, которые очищаются после ввода? 4) Мне кажется при удалении задачи должно появлятся предупреждение, если эта задача имеет подзадачи

В целом мне кажется, что все соответствует ТЗ и тому, что обсуждали

Dary5 commented 1 year ago

@aleksandra-shchegoleva

  1. Информация о приоритете есть в окне "Настройки". Предлагаю выносить как кнопку-переключатель.
  2. Поправила.
  3. Да, логин/пароль сохраняются списком в окне "Настройки".
  4. Включила отдельный вид окна "Предупреждение", которое появляется при конфликте в системе.
aleksandra-shchegoleva commented 1 year ago

@Dary5 @astronik00 Замечаний нет

astronik00 commented 1 year ago

@Dary5 Вроде обсуждали, что изменение приоритета сервера будет осуществляться в настройках, но по твоей диаграмме оно находится на главном экране

astronik00 commented 1 year ago

@Dary5 Исправь картинку на превью MR

Dary5 commented 1 year ago

готово