9 Commits

Author SHA1 Message Date
gorbenko.pavel 948ef76d17 правка
Tests / frontend-test (pull_request) Successful in 12s
2026-08-24 12:10:11 +05:00
gorbenko.pavel 89168024df Merge remote-tracking branch 'origin/main' into initial-fixes
Tests / frontend-test (pull_request) Successful in 13s
# Conflicts:
#	server/src/config.ts
#	server/src/index.ts
2026-08-24 11:48:49 +05:00
gorbenko.pavel f2d4fd1768 fix: захардкодить SHAREPOINT_SITE_URL и SHAREPOINT_LIST_NAME
Убрал требование задавать их через .env на сервере — значения одни
и те же для единственного прод-окружения.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-08-24 11:42:17 +05:00
gorbenko.pavel 07bafde43f feat: отправка формы обращения в SharePoint через api.tnsu.kz
POST /feedback собирает поля формы в столбцы списка (Company резолвится
в название по справочнику, Type/Title — по виду обращения) и создаёт
элемент через api.tnsu.kz /SharePoint/ListItem. Ошибки авторизации и
апстрима логируются в stdout и возвращаются клиенту как 502, чтобы сбой
на проде было видно явно.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-08-24 11:39:20 +05:00
gorbenko.pavel 49bf9b3c4e fix: переименовать вид обращения "Жалоба" в "Обращение"
Обновлены label в KIND_OPTIONS, комментарии и placeholder ФИО в форме.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-08-24 11:13:23 +05:00
gorbenko.pavel e4678210ab Merge remote-tracking branch 'origin/main' into initial-fixes 2026-08-24 09:09:22 +05:00
admin a8d46208a6 добавлена авторизация для запросов на api.tnsu.kz
Tests / frontend-test (pull_request) Successful in 18s
2026-08-21 17:11:34 +05:00
admin 60e654c80e fix: update port mapping for hr-appeals-web service to use 80:80 2026-08-21 16:36:27 +05:00
admin 838fd644e9 Настроен деплой 2026-08-21 15:47:43 +05:00
10 changed files with 161 additions and 18 deletions
+11
View File
@@ -0,0 +1,11 @@
{
"version": "0.0.1",
"configurations": [
{
"name": "hr-appeals-web",
"runtimeExecutable": "npm",
"runtimeArgs": ["run", "dev"],
"port": 3000
}
]
}
+3 -1
View File
@@ -12,5 +12,7 @@ export const config = {
certPath: requireEnv('GRAPH_CERT_PATH'),
keyPath: requireEnv('GRAPH_KEY_PATH'),
scope: process.env.GRAPH_SCOPE ?? 'https://graph.microsoft.com/.default',
port: Number(process.env.PORT ?? 3000)
port: Number(process.env.PORT ?? 3000),
sharepointSiteUrl: 'https://tnsukz.sharepoint.com/sites/hrm',
sharepointListName: 'Обращения'
};
+16
View File
@@ -1,8 +1,10 @@
import express from 'express';
import { config } from './config';
import { getAppOnlyToken } from './graphAuth';
import { submitFeedback } from './sharepoint';
const app = express();
app.use(express.json());
app.get('/healthz', (_req, res) => {
res.status(200).send('ok');
@@ -20,6 +22,20 @@ app.get('/diag/token', async (_req, res) => {
}
});
app.post('/feedback', async (req, res) => {
try {
await submitFeedback(req.body);
res.status(201).json({ ok: true });
} catch (error) {
const message = (error as Error).message;
// Логируем целиком в stdout прод-контейнера — если отвалится авторизация
// (протухший/незарегистрированный сертификат, неверный scope), это должно
// быть видно сразу в логах, а не тонуть в одной строке ответа клиенту.
console.error('Не удалось отправить обращение в SharePoint:', message);
res.status(502).json({ ok: false, error: message });
}
});
app.listen(config.port, () => {
console.log(`hr-appeals-auth listening on :${config.port}`);
});
+91
View File
@@ -0,0 +1,91 @@
import { config } from './config';
import { getAppOnlyToken } from './graphAuth';
const SHAREPOINT_API_URL = 'https://api.tnsu.kz/SharePoint/ListItem';
export type AppealKind = 'complaint' | 'suggestion' | 'other';
const KIND_LABELS: Record<AppealKind, string> = {
complaint: 'Обращение',
suggestion: 'Предложение',
other: 'Другое'
};
// Форма присылает id компании из справочника (см. COMPANY_OPTIONS во фронтовом
// src/features/feedback/model.ts) — колонка Company в SharePoint текстовая и
// ждёт название, а не id, поэтому здесь его резолвим. Список держим в синхроне
// с фронтовым вручную — общего пакета между сервером и клиентом сейчас нет.
const COMPANY_NAMES: Record<string, string> = {
'8028abac-ac23-8fd8-11ef-cb57631a9ab0': 'ТОО AFL Engineering',
'9de064a4-44c8-88ef-11ec-831c058b1141': 'ТОО CL MANAGEMENT',
'9de064a4-44c8-88ef-11ec-8287fb7737a2': 'ТОО KazPromService',
'ec3c78a7-29ef-4725-11ed-a6d8fcd0911d': 'ТОО QAZGEOMAX',
'15009595-005d-00fa-11ef-11309f5e395d': 'ТОО Selin Invest',
'03f4deac-f543-e8cf-11f1-84eb25329fc3': 'ТОО Selin Studio',
'9de064a4-44c8-88ef-11ec-8284930fad96': 'ТОО TANSU Construction',
'9de064a4-44c8-88ef-11ec-8285925d83d2': 'ТОО АСТЕЛ-К',
'15009595-005d-00fa-11ef-0b8bb5ec1a88': 'ТОО ГеоТрек',
'15009595-005d-00fa-11ef-0b8714483372': 'ТОО Монтажник-Т',
'8028beac-ac23-8fd8-11f0-5351706e5996': 'ТОО ОСН Беркут',
'8028b0ac-ac23-8fd8-11ef-dcdb24b82e02': 'ТОО СТ Global',
'15009595-005d-00fa-11ef-1131801d9f3e': 'ТОО ТехЛаб',
'03f4deac-f543-e8cf-11f1-84f51483f59e': 'Частная компания Seline Holding Ltd'
};
export interface IFeedbackSubmission {
company: string;
isAnonymous: boolean;
fullName: string;
position: string;
wantsFeedback: boolean;
kind: AppealKind;
kindOther: string;
situation: string;
suggestion: string;
}
// Имена полей — internal name столбцов списка SharePoint (Company/Name/Position/
// Type/IssueDescription/SuggestionText/WantsFeedback), не отображаемые названия.
function buildFields(data: IFeedbackSubmission): Record<string, unknown> {
const typeLabel = data.kind === 'other' ? data.kindOther : KIND_LABELS[data.kind];
const name = data.isAnonymous ? '' : data.fullName;
const companyName = COMPANY_NAMES[data.company];
if (!companyName) {
console.warn(`Неизвестный id компании из формы, отправляю как есть: ${data.company}`);
}
return {
Title: name ? `${typeLabel} — ${name}` : typeLabel,
Company: companyName ?? data.company,
Name: name,
Position: data.isAnonymous ? '' : data.position,
Type: typeLabel,
IssueDescription: data.situation,
SuggestionText: data.suggestion,
WantsFeedback: data.wantsFeedback
};
}
export async function submitFeedback(data: IFeedbackSubmission): Promise<void> {
const token = await getAppOnlyToken();
const response = await fetch(SHAREPOINT_API_URL, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
Authorization: `Bearer ${token}`
},
body: JSON.stringify({
siteUrl: config.sharepointSiteUrl,
listName: config.sharepointListName,
fields: buildFields(data)
})
});
if (!response.ok) {
throw new Error(
`api.tnsu.kz вернул ошибку при создании элемента списка: ${response.status} ${await response.text()}`
);
}
}
+1 -1
View File
@@ -4,7 +4,7 @@ import { COMPANY_OPTIONS, getAppealFieldsConfig } from './model';
// правило из getAppealFieldsConfig — если оно разойдётся, форма и валидация
// начнут спорить друг с другом. Эти тесты фиксируют контракт правила.
describe('getAppealFieldsConfig', () => {
it('жалоба: описание обязательно, предложение показано, но необязательно', () => {
it('обращение: описание обязательно, предложение показано, но необязательно', () => {
const fields = getAppealFieldsConfig('complaint');
expect(fields.situation).toEqual({ visible: true, required: true });
expect(fields.suggestion).toEqual({ visible: true, required: false });
+2 -2
View File
@@ -43,7 +43,7 @@ export const COMPANY_OPTIONS: ISelectOption[] = [
// значением ниже (EMPTY_FEEDBACK.kind), так что выбор пользователя.
export const KIND_OPTIONS: ISelectOption[] = [
{ value: 'suggestion', label: 'Предложение' },
{ value: 'complaint', label: 'Жалоба' },
{ value: 'complaint', label: 'Обращение' },
{ value: 'other', label: 'Другое' }
];
@@ -61,7 +61,7 @@ export interface IAppealFieldsConfig {
/**
* Состав полей зависит от вида обращения:
* жалоба — обязательно описание, а предложение — на желание автора;
* обращение — обязательно описание, а предложение — на желание автора;
* предложение — только само предложение, «ситуации» ещё нет;
* другое — свободное описание, без предложения.
* Одно правило на форму и на валидацию, чтобы они не разошлись.
+3 -2
View File
@@ -28,7 +28,7 @@ export const TEXTS = {
placeholders: {
company: 'Выберите компанию',
fullName: 'Иванов Иван Иванович',
fullName: 'Нурлан Нурланович Нурланов',
position: 'Например: инженер-проектировщик',
kindOther: 'Свой вариант',
situation: 'Опишите, что произошло: когда, где, кто участвовал',
@@ -46,7 +46,8 @@ export const TEXTS = {
required: 'Заполните поле',
selectRequired: 'Выберите значение',
tooShort: 'Слишком короткий ответ — опишите подробнее',
hasErrors: 'Проверьте выделенные поля — что-то заполнено не полностью.'
hasErrors: 'Проверьте выделенные поля — что-то заполнено не полностью.',
submitFailed: 'Не удалось отправить обращение. Попробуйте ещё раз позже.'
},
thanks: {
+19 -6
View File
@@ -18,8 +18,8 @@ export interface IFeedbackForm {
/**
* Состояние формы обращения: данные, валидация, отправка.
* Бэкенда пока нет — submit только имитирует отправку (см. TODO ниже),
* поэтому вся логика синхронная и живёт здесь, без сервисного слоя.
* Отправка идёт через nginx (/api/... -> hr-appeals-auth) — сервис сам
* меняет её на app-only токен и создаёт элемент в списке SharePoint.
*/
export function useFeedbackForm(): IFeedbackForm {
const [data, setData] = useState<IFeedbackData>(EMPTY_FEEDBACK);
@@ -53,11 +53,24 @@ export function useFeedbackForm(): IFeedbackForm {
setFormError('');
setStatus('submitting');
// TODO: заменить на реальную отправку, когда определимся с бэкендом.
// Сейчас это заглушка по договорённости: делаем только UI.
// eslint-disable-next-line no-console
console.info('Обращение (заглушка отправки):', data);
fetch('/api/feedback', {
method: 'POST',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify(data)
})
.then(async (response) => {
if (!response.ok) {
const body = await response.json().catch(() => null);
throw new Error(body?.error ?? `Сервер вернул ${response.status}`);
}
setStatus('done');
})
.catch((error) => {
// eslint-disable-next-line no-console
console.error('Не удалось отправить обращение:', error);
setFormError(TEXTS.errors.submitFailed);
setStatus('idle');
});
}, [data]);
const reset = useCallback((): void => {
@@ -6,7 +6,7 @@ const VALID_COMPANY = '9de064a4-44c8-88ef-11ec-8285925d83d2';
const LONG_TEXT = 'Развёрнутое описание, которого достаточно для проверки правила';
const SHORT_TEXT = 'коротко';
/** Полностью заполненное «жалоба»-обращение — стартовая точка для тестов. */
/** Полностью заполненное обращение — стартовая точка для тестов. */
function buildValidComplaint(overrides: Partial<IFeedbackData> = {}): IFeedbackData {
return {
...EMPTY_FEEDBACK,
@@ -61,17 +61,17 @@ describe('validateFeedback: вид обращения "Другое"', () => {
// Состав обязательных полей зависит от вида обращения (см. getAppealFieldsConfig
// в model.ts) — эти тесты фиксируют контракт, чтобы форма и валидация не разошлись.
describe('validateFeedback: обязательность полей по видам обращения', () => {
it('жалоба: описание ситуации обязательно', () => {
it('обращение: описание ситуации обязательно', () => {
const errors = validateFeedback(buildValidComplaint({ situation: '' }));
expect(errors.situation).toBeDefined();
});
it('жалоба: предложение по улучшению НЕ обязательно', () => {
it('обращение: предложение по улучшению НЕ обязательно', () => {
const errors = validateFeedback(buildValidComplaint({ suggestion: '' }));
expect(errors.suggestion).toBeUndefined();
});
it('жалоба: если предложение всё же указано, но слишком коротко — ошибка', () => {
it('обращение: если предложение всё же указано, но слишком коротко — ошибка', () => {
const errors = validateFeedback(buildValidComplaint({ suggestion: SHORT_TEXT }));
expect(errors.suggestion).toBeDefined();
});
+10 -1
View File
@@ -12,7 +12,16 @@ export default defineConfig({
},
server: {
port: 3000,
open: true
open: true,
// Как nginx в проде (nginx.conf): /api/... идёт на hr-appeals-auth без
// префикса. Локально сервис запускаем на 3100 — порт 3000 занят самим vite.
proxy: {
'/api': {
target: 'http://localhost:3100',
changeOrigin: true,
rewrite: (path) => path.replace(/^\/api/, '')
}
}
},
test: {
// globals: true — describe/it/expect доступны без импорта, как привычно по Jest.