Conversation
|
|
||
| while (nodes.Count > 0) | ||
| { | ||
| var childrensNode = new List<DirRX.Tree.Structures.Module.ITask>(); |
There was a problem hiding this comment.
тут наверно лучше childrenNodes назвать? и ниже просто children, т.к это уже множественное число, s лишняя
| /// Метод возвращает структуру для формирования дерева отображения задач на исполнение поручения. | ||
| /// </summary> | ||
| /// <param name="id">Ид задачи на исполнение поручения.</param> | ||
| /// <returns>Возвращает структуру для формирования дерева.</returns> |
There was a problem hiding this comment.
Тут и в методах ниже: вроде обычно пишется конкретно, что возвращается, типа "Структура для формирования дерева.", потому что в подсказке уже будет слово Returns (Возврат)
| private DirRX.Tree.Structures.Module.ITask CreateTreeNode(DirRX.TreeViewer.IActionItemExecutionTask task) | ||
| { | ||
| var node = DirRX.Tree.Structures.Module.Task.Create(); | ||
| node.name = Sungero.Docflow.PublicFunctions.Module.CutText(task.Subject, 65); |
There was a problem hiding this comment.
Давай число сразу в константы уберем и назовем информативно, а то непонятно, что за магическое 65
There was a problem hiding this comment.
вынес в константы
| 1. Сокращенная тема задачи. Оформлена в виде ссылки для удобного перехода в карточку задачи. | ||
| 2. ФИО исполнителя. | ||
| 3. Срок задачи. | ||
| 4. Обводка узла: |
There was a problem hiding this comment.
Может вместо "обводка" - "рамка" / "граница" / "контур"? в ворде/экселе слова граница и контур используются
There was a problem hiding this comment.
поменял на контур
| <img width="1022" height="799" alt="image" src="https://github.com/user-attachments/assets/cfa7f6f1-4e8d-42b5-a11f-b9f55e6fbe5a" /> | ||
|
|
||
|
|
||
| Перед началом работы с деревом, необходимо настроить представление формы для Задачи на исполнение поручения. Для этого необходимо добавить в карточку новую вкладку и разместить на ней специальный контрол "Дерево". Пример ниже на скриншоте: |
There was a problem hiding this comment.
Опечатка: запятая лишняя, "Перед началом работы с деревом необходимо настроить представление формы"
| 3. Установите пакеты проекта, для этого в командную строку среды разработки введите команду "**npm I**". Это также можно сделать и через обычную командную строку системы, перейдя в каталог с проектом. | ||
| 4. Для отладки стороннего контрола используйте команду "**npm run start:dev:standalone**". После этого перейдите по адресу http://localhost:3001/. | ||
| > [!NOTE] | ||
| > При отладке, потребуются данные для построения дерева. Вариант временной доработки для получения данных: |
There was a problem hiding this comment.
Лишняя запятая после "При отладке"
| > При отладке, потребуются данные для построения дерева. Вариант временной доработки для получения данных: | ||
| > - В методе **ExecuteGetRequestAsync** добавить в headers способ авторизации; | ||
| > - В методе **GetAllActionItemExecutionTaskForLeadTaskAsync** указать ид конкретной задачи. | ||
| > - В config.yml у Directum RX укажите в разделе сервиса интеграции адрес http://localhost:3001/ в **CORS_ALLOWED_ORIGINS_LIST**, чтобы не возникали проблемы с CORS политиками. Подробности в [справке](https://club.directum.ru/webhelp/directumrx/web/index.html?admin_config_integrationservice_linux.htm). |
There was a problem hiding this comment.
лучше "в параметре (или переменной) CORS_ALLOWED_ORIGINS_LIST"
| 4. Для отладки стороннего контрола используйте команду "**npm run start:dev:standalone**". После этого перейдите по адресу http://localhost:3001/. | ||
| > [!NOTE] | ||
| > При отладке, потребуются данные для построения дерева. Вариант временной доработки для получения данных: | ||
| > - В методе **ExecuteGetRequestAsync** добавить в headers способ авторизации; |
There was a problem hiding this comment.
- А получится тут пример дописать?
- Может в явную указать, что методы искать в js-коде?
There was a problem hiding this comment.
- Привел пример
- Выше идет раздел Компоненты для сборки и разработки стороннего контрола, он уже подразумевает, что речь идет про js/ts код
| @@ -1 +1,117 @@ | |||
| # rx-template-actionitem-tree-view | |||
| # Отображение задач на исполнение поручения в виде дерева | |||
There was a problem hiding this comment.
Давай назовем "Пример разработки контрола: дерево задач на исполнение поручения"
Потому что к ШР переходят с разводящей страницы, из названия кейса неочевидно, что ШР можно использовать как пример, хотелось бы в заголовке и то, что это пример, отразить, и сам кейс, если нужен именно он
| > | ||
| > Внимание! Изменения будут вноситься только в новые версии. | ||
|
|
||
| ## Описание |
There was a problem hiding this comment.
А можем в явном виде как-то сформулировать (может быть в виде примечания), что решение можно использовать в качестве примера для разработки своих контролов?
There was a problem hiding this comment.
добавил примечание
| @@ -1 +1,136 @@ | |||
| # rx-template-actionitem-tree-view | |||
| # Пример разработки контрола: дерево задач на исполнение поручения | |||
| Репозиторий с шаблоном разработки "Отображение задач на исполнение поручения в виде дерева". | |||
There was a problem hiding this comment.
тут осталось старое название репоза
| Репозиторий с шаблоном разработки "Отображение задач на исполнение поручения в виде дерева". | ||
| > [!NOTE] | ||
| > Замечания и пожелания по развитию шаблона разработки фиксируйте через [Issues](https://github.com/DirectumCompany/rx-template-actionitem-tree-view/issues). | ||
| При оформлении ошибки, опишите сценарий для воспроизведения. Для пожеланий приведите обоснование для описываемых изменений - частоту использования, бизнес-ценность, риски и/или эффект от реализации. |
There was a problem hiding this comment.
"При оформлении ошибки, опишите" - тут лишняя запятая
| В состав решения входит: | ||
| 1. Перекрытие задачи на исполнение поручения (**ActionItemExecutionTask**). | ||
| 2. В задачу добавлено новое свойство **SuperiorTaskDirRX**. В нем заполняется ссылка на родительскую задачу на исполнение поручения. | ||
| 3. Добавлен новый модуль **Tree**. В нем реализован API **GetTreeStructureActionItemExecutionTask**, которое передает данные для постороения дерева в стороннем контроле. |
There was a problem hiding this comment.
тут осталась нестыковка "API реализован", но "api, которое"
по идее дальше идет название функции, может быть переформулировать как-то "В нем реализована функция интеграции, которая возвращает данные для построения дерева в стороннем контроле."?
There was a problem hiding this comment.
поправил + дописал название функции
|
|
||
| В состав решения входит: | ||
| 1. Перекрытие задачи на исполнение поручения (**ActionItemExecutionTask**). | ||
| 2. В задачу добавлено новое свойство **SuperiorTaskDirRX**. В нем заполняется ссылка на родительскую задачу на исполнение поручения. |
There was a problem hiding this comment.
SuperiorTaskDirRX - у нас теперь SuperiorTask
| >``` | ||
| > - В методе **GetAllActionItemExecutionTaskForLeadTaskAsync** указать ид конкретной задачи. | ||
| > - В config.yml у Directum RX укажите в разделе сервиса интеграции адрес http://localhost:3001/ в параметре **CORS_ALLOWED_ORIGINS_LIST**, чтобы не возникали проблемы с CORS политиками. Подробности в [справке](https://club.directum.ru/webhelp/directumrx/web/index.html?admin_config_integrationservice_linux.htm). | ||
| 5. Для сборки стороннего контрола используйте команду "**npm run build:release**". В результате в папке dist будет создан ряд файлов для загрузки в DDS. |
There was a problem hiding this comment.
"в DDS." - лучше написать "среде разработки Directum RX", если еще где-то есть, то тоже
| > async function ExecuteGetRequestAsync(url: string) { | ||
| > const response = await fetch(url, { | ||
| > headers: { | ||
| > 'Authorization': 'Basic ' + btoa('Administrator:11111') |
There was a problem hiding this comment.
примере лучше вместо "11111" написать
|
|
||
| Ниже представлен пример отображения задач на исполнение поручения в виде дерева. | ||
|
|
||
| <img width="1022" height="799" alt="image" src="https://github.com/user-attachments/assets/cfa7f6f1-4e8d-42b5-a11f-b9f55e6fbe5a" /> |
There was a problem hiding this comment.
А получится сделать более ветвистое дерево для скрина? Если нет, то давай выровняем, чтобы дерево по центру скрина было
| 5. Для сборки стороннего контрола используйте команду "**npm run build:release**". В результате в папке dist будет создан ряд файлов для загрузки в DDS. | ||
|
|
||
| ## Варианты расширения функциональности на проектах | ||
| 1. Расширение структуры для передачи дополнительных данных в сторонний контрол. |
There was a problem hiding this comment.
Тут получится прописать, что за структура имеется в виду? Может быть сразу с примером типа "например, добавить передачу свойств таких-то"
|
|
||
| В состав решения входит: | ||
| 1. Перекрытие задачи на исполнение поручения (**ActionItemExecutionTask**). | ||
| 2. В задачу добавлено новое свойство **SuperiorTask**. В нем заполняется ссылка на родительскую задачу на исполнение поручения. |
There was a problem hiding this comment.
Тут можно подсветить, почему именно новое св-во, а не mainTask
| ## Варианты расширения функциональности на проектах | ||
| 1. Расширение структуры для передачи дополнительных данных в сторонний контрол. | ||
| 2. Доработка узла **NodeContent** для отображения дополнительной информации по задаче или изменения дизайна. | ||
| 3. Реализация логики с вертикальным отображением. JSX объект **Tree** принимает в качестве параметра ориентацию дерева (**orientation**). |
There was a problem hiding this comment.
вот тут еще подробнее бы сформулировать
No description provided.