@escapenavigator/services 1.10.139 → 1.10.140

package/dist/api/init-client-api.d.ts

@@ -71,6 +71,7 @@ import { uploadsApiDeclaration } from './uploads-api';
 import { upsellingsApiDeclaration } from './upsellings-api';
 import { userMessagesApiDeclaration } from './user-messages-api';
 import { usersApiDeclaration } from './users-api';
+import { widgetSessionsApiDeclaration } from './widget-sessions-api';
 import { widgetsApiDeclaration } from './widgets-api';
 export type InitApiDeclaration = {
     customFieldsApi: Api<typeof customFieldsApiDeclaration>;
@@ -129,6 +130,7 @@ export type InitApiDeclaration = {
     partners: Api<typeof partnersApiDeclaration>;
     questrooms: Api<typeof questroomsApiDeclaration>;
     widgets: Api<typeof widgetsApiDeclaration>;
+    widgetSessions: Api<typeof widgetSessionsApiDeclaration>;
     profiles: Api<typeof profilesApiDeclaration>;
     users: Api<typeof usersApiDeclaration>;
     clients: Api<typeof clientsApiDeclaration>;

package/dist/api/init-client-api.js

@@ -75,6 +75,7 @@ var uploads_api_1 = require("./uploads-api");
 var upsellings_api_1 = require("./upsellings-api");
 var user_messages_api_1 = require("./user-messages-api");
 var users_api_1 = require("./users-api");
+var widget_sessions_api_1 = require("./widget-sessions-api");
 var widgets_api_1 = require("./widgets-api");
 var initClientApi = function (url, clientOptions) { return ({
     customFieldsApi: (0, _1.initApi)(custom_fields_api_1.customFieldsApiDeclaration, url, clientOptions),
@@ -133,6 +134,7 @@ var initClientApi = function (url, clientOptions) { return ({
     partners: (0, _1.initApi)(partners_api_1.partnersApiDeclaration, url, clientOptions),
     questrooms: (0, _1.initApi)(questrooms_api_1.questroomsApiDeclaration, url, clientOptions),
     widgets: (0, _1.initApi)(widgets_api_1.widgetsApiDeclaration, url, clientOptions),
+    widgetSessions: (0, _1.initApi)(widget_sessions_api_1.widgetSessionsApiDeclaration, url, clientOptions),
     profiles: (0, _1.initApi)(profiles_api_1.profilesApiDeclaration, url, clientOptions),
     users: (0, _1.initApi)(users_api_1.usersApiDeclaration, url, clientOptions),
     clients: (0, _1.initApi)(clients_api_1.clientsApiDeclaration, url, clientOptions),

package/dist/api/widget-sessions-api/get-full.d.ts

@@ -0,0 +1,3 @@
+import { WidgetSessionEventsRO } from '@escapenavigator/types/dist/widget-sessions/widget-session-events.ro';
+import { ApiMethodDeclaration } from '..';
+export declare const getFull: ApiMethodDeclaration<number, WidgetSessionEventsRO>;

package/dist/api/widget-sessions-api/get-full.js

@@ -0,0 +1,8 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.getFull = void 0;
+var getFull = function (id) { return ({
+    url: "/admin/widget-sessions/".concat(id, "/full"),
+    method: 'GET',
+}); };
+exports.getFull = getFull;

package/dist/api/widget-sessions-api/index.d.ts

@@ -0,0 +1,12 @@
+import { getFull } from './get-full';
+import { list } from './list';
+import { remove } from './remove';
+import { updateComment } from './update-comment';
+type ApiDeclaration = {
+    list: typeof list;
+    getFull: typeof getFull;
+    updateComment: typeof updateComment;
+    remove: typeof remove;
+};
+export declare const widgetSessionsApiDeclaration: ApiDeclaration;
+export {};

package/dist/api/widget-sessions-api/index.js

@@ -0,0 +1,13 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.widgetSessionsApiDeclaration = void 0;
+var get_full_1 = require("./get-full");
+var list_1 = require("./list");
+var remove_1 = require("./remove");
+var update_comment_1 = require("./update-comment");
+exports.widgetSessionsApiDeclaration = {
+    list: list_1.list,
+    getFull: get_full_1.getFull,
+    updateComment: update_comment_1.updateComment,
+    remove: remove_1.remove,
+};

package/dist/api/widget-sessions-api/list.d.ts

@@ -0,0 +1,12 @@
+import { QueryDto } from '@escapenavigator/types/dist/shared/query.dto';
+import { QueryRO } from '@escapenavigator/types/dist/shared/query.ro';
+import { WidgetSessionRO } from '@escapenavigator/types/dist/widget-sessions/widget-session.ro';
+import { ApiMethodDeclaration } from '..';
+type ResponseData = QueryRO<WidgetSessionRO & {
+    profile?: {
+        id: number;
+        title: string;
+    };
+}>;
+export declare const list: ApiMethodDeclaration<QueryDto, ResponseData>;
+export {};

package/dist/api/widget-sessions-api/list.js

@@ -0,0 +1,9 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.list = void 0;
+var list = function (params) { return ({
+    url: '/admin/widget-sessions',
+    method: 'GET',
+    params: params,
+}); };
+exports.list = list;

package/dist/api/widget-sessions-api/remove.d.ts

@@ -0,0 +1,4 @@
+import { ApiMethodDeclaration } from '..';
+export declare const remove: ApiMethodDeclaration<number, {
+    success: boolean;
+}>;

package/dist/api/widget-sessions-api/remove.js

@@ -0,0 +1,8 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.remove = void 0;
+var remove = function (id) { return ({
+    url: "/admin/widget-sessions/".concat(id),
+    method: 'DELETE',
+}); };
+exports.remove = remove;

package/dist/api/widget-sessions-api/update-comment.d.ts

@@ -0,0 +1,9 @@
+import { ApiMethodDeclaration } from '..';
+type Params = {
+    id: number;
+    comment?: string;
+};
+export declare const updateComment: ApiMethodDeclaration<Params, {
+    success: boolean;
+}>;
+export {};

package/dist/api/widget-sessions-api/update-comment.js

@@ -0,0 +1,12 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.updateComment = void 0;
+var updateComment = function (_a) {
+    var id = _a.id, comment = _a.comment;
+    return ({
+        url: "/admin/widget-sessions/".concat(id),
+        method: 'PATCH',
+        data: { comment: comment },
+    });
+};
+exports.updateComment = updateComment;

package/dist/hooks/use-api-method/index.js

@@ -48,8 +48,33 @@ var api_2 = require("./reducers/api");
 var useApiMethod = function (_a) {
     var _b;
     var api = _a.api, successCallback = _a.successCallback, errorCallback = _a.errorCallback;
-    var mounted = (0, react_1.useRef)(false);
+    // mounted стартует как `true`, а не `false`. Прежнее `useRef(false)` имело
+    // тонкий баг: между первым render'ом и первым flush-ем useEffect'а
+    // mounted=false, и если в этот микро-промежуток успевал свалиться запрос
+    // (быстрая сеть, кэш, microtask до commit'а) — catch-блок видел mounted=false
+    // и молча возвращал null без throw → консьюмер думал «всё ок», state
+    // оставался в loading навсегда. В React 18 StrictMode dev двойной mount
+    // только усугублял проблему (cleanup первой пары ставил false, и до
+    // useEffect второй пары mounted был false). Правда жизни: компонент уже
+    // смонтирован к моменту, как мы вообще можем вызвать fetch — значение
+    // должно быть true с самого начала, а cleanup сам поставит false.
+    var mounted = (0, react_1.useRef)(true);
     var source = (0, react_1.useRef)(null);
+    // Inflight-counter для устойчивого loading-state в кейсе нескольких
+    // sequential/parallel вызовов на одном хук-инстансе. Без него каждый
+    // успех/фейл одной итерации делал dispatch(fetchSuccess|Failure) с
+    // loading=false → между итерациями кнопка мигала enabled-состоянием
+    // (риск двойного клика), скелетоны таблиц моргали. Считаем
+    // активные fetch'и и dispatch'им «loading=false» только когда счётчик
+    // вернулся в 0. На реальную семантику data/error не влияет — последний
+    // успех всё равно перетирает state.data, последняя ошибка — state.error.
+    //
+    // ВАЖНО про cancellation: useApiMethod исторически отменяет предыдущий
+    // запрос при новом fetch() (single source.current). Из-за этого Promise.all
+    // на одном инстансе ломается — все, кроме последнего вызова, возвращают
+    // null. Inflight-counter эту семантику не меняет, только сглаживает
+    // флик loading-state между быстрыми последовательными fetch'ами.
+    var inflightRef = (0, react_1.useRef)(0);
     var _c = (0, react_1.useReducer)((api_2.apiReducer), api_2.initialState), state = _c[0], dispatch = _c[1];
     // КРИТИЧНО: коллбэки держим в ref, а не в deps useCallback. Иначе
     // call-site вроде `useApiMethod({ api, successCallback: () => ... })`
@@ -63,25 +88,36 @@ var useApiMethod = function (_a) {
     successCallbackRef.current = successCallback;
     errorCallbackRef.current = errorCallback;
     var fetch = (0, react_1.useCallback)(function (params) { return __awaiter(void 0, void 0, void 0, function () {
-        var response, error_1, text, _a, err;
+        var wasIdle, response, error_1, text, _a, err;
         var _b, _c, _d, _e;
         return __generator(this, function (_f) {
             switch (_f.label) {
                 case 0:
                     (_b = source.current) === null || _b === void 0 ? void 0 : _b.cancel();
                     source.current = axios_1.default.CancelToken.source();
+                    wasIdle = inflightRef.current === 0;
+                    inflightRef.current += 1;
                     _f.label = 1;
                 case 1:
-                    _f.trys.push([1, 3, , 8]);
-                    dispatch((0, api_1.fetchStart)());
+                    _f.trys.push([1, 3, 8, 9]);
+                    if (wasIdle)
+                        dispatch((0, api_1.fetchStart)());
                     return [4 /*yield*/, api(params, {
                             cancelToken: source.current.token,
                         })];
                 case 2:
                     response = _f.sent();
-                    if (successCallbackRef.current)
-                        successCallbackRef.current(response, params);
-                    dispatch((0, api_1.fetchSuccess)(response));
+                    // Mounted-guard на success-path. Раньше успешный response
+                    // после unmount всё равно дёргал successCallback (мог сделать
+                    // navigate / showToast на размонтированном UI) и dispatch на
+                    // мёртвый reducer (React 18 warning + лишний state-change в
+                    // никуда). Теперь после unmount — тихо отдаём response наверх,
+                    // и пусть консьюмер сам решает, что с ним делать.
+                    if (mounted.current) {
+                        if (successCallbackRef.current)
+                            successCallbackRef.current(response, params);
+                        dispatch((0, api_1.fetchSuccess)(response));
+                    }
                     return [2 /*return*/, response];
                 case 3:
                     error_1 = _f.sent();
@@ -112,7 +148,10 @@ var useApiMethod = function (_a) {
                     dispatch((0, api_1.fetchFailure)(err));
                     // eslint-disable-next-line @typescript-eslint/no-throw-literal
                     throw err;
-                case 8: return [2 /*return*/];
+                case 8:
+                    inflightRef.current = Math.max(0, inflightRef.current - 1);
+                    return [7 /*endfinally*/];
+                case 9: return [2 /*return*/];
             }
         });
     }); }, [api]);

package/dist/hooks/use-api-method/reducers/api.js

@@ -24,7 +24,17 @@ var apiReducer = function (state, action) {
     if (state === void 0) { state = exports.initialState; }
     switch (action.type) {
         case api_1.ActionTypes.FETCH_START:
-            return __assign(__assign({}, exports.initialState), { loading: true });
+            // ВАЖНО: НЕ сбрасываем `data`/`response`/`statusOk` — это
+            // stale-while-revalidate. Прежнее `return { ...initialState, loading: true }`
+            // обнуляло данные на каждый ре-фетч, и любой консьюмер
+            // (таблицы списков, шапка с метой ордера, расписание) на
+            // время запроса показывал пустое состояние / скелетон, даже
+            // если новые данные через 200мс прилетят те же самые.
+            // Теперь данные сохраняются до прихода нового ответа —
+            // skeleton flicker на ре-фетчах исчезает. Только `error`
+            // ресетим, чтобы старая ошибка не «сидела» поверх нового
+            // запроса (если новый успешен, она и так заменится).
+            return __assign(__assign({}, state), { loading: true, error: null });
         case api_1.ActionTypes.FETCH_SUCCESS:
             return __assign(__assign({}, state), { loading: false, data: action.payload.response.data, response: action.payload.response, statusOk: true });
         case api_1.ActionTypes.FETCH_FAILURE:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@escapenavigator/services",
-  "version": "1.10.139",
+  "version": "1.10.140",
   "publishConfig": {
     "access": "public"
   },
@@ -12,10 +12,10 @@
   "scripts": {
     "build": "rm -rf dist && tsc --project tsconfig.json"
   },
-  "gitHead": "1b34f498be3d6fb27dccdd2a3663cef5169bbc91",
+  "gitHead": "cb2f3ab38ef5b8b2a38ab5b2c485e484a0cf9414",
   "dependencies": {
-    "@escapenavigator/types": "^1.10.130",
-    "@escapenavigator/utils": "^1.10.134",
+    "@escapenavigator/types": "^1.10.131",
+    "@escapenavigator/utils": "^1.10.135",
     "axios": "^0.21.4",
     "class-transformer": "^0.5.1",
     "class-validator": "^0.13.2",