сторонний контрол для отображения дерева задач by vadimpain · Pull Request #2 · DirectumCompany/rx-template-actionitem-tree-view

vadimpain · 2026-02-12T12:53:22Z

No description provided.

alla100lnikova · 2026-02-18T18:39:26Z

@@ -0,0 +1,10 @@
+# Introduction 
+Пример сторонних контролов для вебклиента с использованием Webpack Module Federation и React


опечатка? веб-клиента

поправил

alla100lnikova · 2026-02-19T10:34:14Z

+      name: 'Tree',
+      // Загрузчики контрола. Загрузчик - это функция, принимающая информацию о контексте и точку доступа к API.
+      // Задача загрузчика - смонтировать корневой UI-компонент контрола в указанный DOM-элемент.
+      // Один и тот же контрол может отображаться в разных контекстах (карточка и обложка модуля). Если контрол должен отображаться и в карточке и в обложке, 


пропущена запятая: и в карточке, и в обложке

поправил

alla100lnikova · 2026-02-19T10:34:31Z

+      // то для него следует создать разные загрузчики для этих контекстов (*-cover-loader и *-card-loader).
+      loaders: [
+        {
+          // Имя загручика должно соответствовать имени загрузчика в файле component.loaders.ts.


опечатка: загрузчика

поправил

iam-ktn

В целом ничего криминального нет, есть несколько рекомендаций
Платформа рекомендует webpack использовать? Если будет время можно попробовать перевести на vite - сильно быстрее будет собираться
Можно попробовать использовать библиотеку стилей (но это так, в качестве упражнения)

iam-ktn · 2026-04-02T07:26:12Z

@@ -0,0 +1,25 @@
+{
+    "gantt": {


@vadimpain Локализация, да и сам пакет не используется? Лучше убрать

iam-ktn · 2026-04-02T07:26:34Z

@vadimpain

.ts?

По разделению файлов. У тебя есть 2 ответственности в файле сейчас - базовые REST-запросы и специальная логика для получения дерева поручений. Предлагаю такую структуру:

/src/utils/request.ts // Тут будет базовый метод /src/action-items/action-items.client.ts // Тут запросы к конкретным endpoint-ам

добавить типизацию

iam-ktn · 2026-04-02T07:27:13Z

+    return response.json();
+ }
+
+ function executeGetRequest(url: string, async: boolean = false) {


@vadimpain Предлагаю оставить только асинхронные вызовы

iam-ktn · 2026-04-02T07:27:21Z

+    return request;
+}
+
+export function GetAllActionItemExecutionTaskForLeadTask(taskId: number) {


@vadimpain camelCase. И в следующем методе тоже

iam-ktn · 2026-04-02T07:28:26Z

@@ -0,0 +1,43 @@
+export const GetIntServiceUrl = () => window.location.origin + "/integration/OData/";


@vadimpain Лучше baseUrl хранить в конфиге ну или в крайнем случае запрашивать отдельным get-ом. Есть какие-то рекомендации от платформы?

iam-ktn · 2026-04-02T07:31:22Z

+      const response = await executeGetRequestAsync(url);
+      console.log(response);
+      return response.value;
+    } catch (error) {


@vadimpain NBD: потом желательно обработку ошибок сделать визуальной - через попапы (как в самой системе). И тут же тоже могут быть разные причины (сетевая проблемы, в ответе пришел не JSON и т.п.)

iam-ktn · 2026-04-02T07:33:45Z

@vadimpain Предлагаю обсудить стилизацию. Можно попробовать какой-нибудь фреймворк, где стили встроены в код компоненты - antd, MUI

iam-ktn · 2026-04-02T07:46:54Z

+};
+
+// Компоненты
+const NodeContent: React.FC<{


@vadimpain Давай вынесем в отдельную компоненту?

iam-ktn · 2026-04-02T07:48:03Z

+type StatusKey = 'Completed' | 'InProcess' | 'InProcessCurrent' | 'Aborted';
+
+// Конфигурация стилей и иконок для статусов
+const STATUS_CONFIG = {


@vadimpain Map<StatusKey, { border: string; icon: string | null }> ?

iam-ktn · 2026-04-02T07:50:18Z

+} as const satisfies Record<StatusKey, { border: string; icon: string | null }>;
+
+// Хуки и утилиты
+const useTreeData = (entityId: number) => {


@vadimpain NBD: Для читаемости можно вынести в store. Тогда в коде загрузка будет выглядеть как-то так:

useEffect(() => { itemsStore.loadItems(entityId) }, [entityId]); ... if (itemsStore.isLoading) return <Spin />; // Или другой лоадер itemsStore.data.forEach(item => { // Render })

Сейчас, конечно, выгода не очевидна этого. Давай обсудим

iam-ktn · 2026-04-03T11:22:04Z

+  const [isLoading, setIsLoading] = useState(false);
+
+  const fetchData = useCallback(async () => {
+    setIsLoading(true);


@vadimpain Еще тут бы хорошо делать setError(null); Сейчас этот стейт нигде не сбрасывается

Vadim Pchelnikov added 2 commits February 12, 2026 16:51

сторонний контрол для отображения дерева задач

cd1041b

правка

b1b9948

vadimpain self-assigned this Feb 13, 2026

vadimpain requested a review from alla100lnikova February 13, 2026 11:23

alla100lnikova reviewed Feb 19, 2026

View reviewed changes

правки после ревью

a801ed3

alla100lnikova approved these changes Apr 1, 2026

View reviewed changes

vadimpain requested a review from iamktn April 2, 2026 05:32

iam-ktn reviewed Apr 2, 2026

View reviewed changes

iam-ktn reviewed Apr 3, 2026

View reviewed changes

		@@ -0,0 +1,10 @@
		# Introduction
		Пример сторонних контролов для вебклиента с использованием Webpack Module Federation и React

		@@ -0,0 +1,43 @@
		export const GetIntServiceUrl = () => window.location.origin + "/integration/OData/";

Conversation

vadimpain commented Feb 12, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

iam-ktn left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants