Compare commits
9 Commits
main
...
948ef76d17
| Author | SHA1 | Date | |
|---|---|---|---|
| 948ef76d17 | |||
| 89168024df | |||
| f2d4fd1768 | |||
| 07bafde43f | |||
| 49bf9b3c4e | |||
| e4678210ab | |||
| a8d46208a6 | |||
| 60e654c80e | |||
| 838fd644e9 |
@@ -0,0 +1,11 @@
|
|||||||
|
{
|
||||||
|
"version": "0.0.1",
|
||||||
|
"configurations": [
|
||||||
|
{
|
||||||
|
"name": "hr-appeals-web",
|
||||||
|
"runtimeExecutable": "npm",
|
||||||
|
"runtimeArgs": ["run", "dev"],
|
||||||
|
"port": 3000
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
@@ -12,5 +12,7 @@ export const config = {
|
|||||||
certPath: requireEnv('GRAPH_CERT_PATH'),
|
certPath: requireEnv('GRAPH_CERT_PATH'),
|
||||||
keyPath: requireEnv('GRAPH_KEY_PATH'),
|
keyPath: requireEnv('GRAPH_KEY_PATH'),
|
||||||
scope: process.env.GRAPH_SCOPE ?? 'https://graph.microsoft.com/.default',
|
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: 'Обращения'
|
||||||
};
|
};
|
||||||
|
|||||||
@@ -1,8 +1,10 @@
|
|||||||
import express from 'express';
|
import express from 'express';
|
||||||
import { config } from './config';
|
import { config } from './config';
|
||||||
import { getAppOnlyToken } from './graphAuth';
|
import { getAppOnlyToken } from './graphAuth';
|
||||||
|
import { submitFeedback } from './sharepoint';
|
||||||
|
|
||||||
const app = express();
|
const app = express();
|
||||||
|
app.use(express.json());
|
||||||
|
|
||||||
app.get('/healthz', (_req, res) => {
|
app.get('/healthz', (_req, res) => {
|
||||||
res.status(200).send('ok');
|
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, () => {
|
app.listen(config.port, () => {
|
||||||
console.log(`hr-appeals-auth listening on :${config.port}`);
|
console.log(`hr-appeals-auth listening on :${config.port}`);
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -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()}`
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -4,7 +4,7 @@ import { COMPANY_OPTIONS, getAppealFieldsConfig } from './model';
|
|||||||
// правило из getAppealFieldsConfig — если оно разойдётся, форма и валидация
|
// правило из getAppealFieldsConfig — если оно разойдётся, форма и валидация
|
||||||
// начнут спорить друг с другом. Эти тесты фиксируют контракт правила.
|
// начнут спорить друг с другом. Эти тесты фиксируют контракт правила.
|
||||||
describe('getAppealFieldsConfig', () => {
|
describe('getAppealFieldsConfig', () => {
|
||||||
it('жалоба: описание обязательно, предложение показано, но необязательно', () => {
|
it('обращение: описание обязательно, предложение показано, но необязательно', () => {
|
||||||
const fields = getAppealFieldsConfig('complaint');
|
const fields = getAppealFieldsConfig('complaint');
|
||||||
expect(fields.situation).toEqual({ visible: true, required: true });
|
expect(fields.situation).toEqual({ visible: true, required: true });
|
||||||
expect(fields.suggestion).toEqual({ visible: true, required: false });
|
expect(fields.suggestion).toEqual({ visible: true, required: false });
|
||||||
|
|||||||
@@ -43,7 +43,7 @@ export const COMPANY_OPTIONS: ISelectOption[] = [
|
|||||||
// значением ниже (EMPTY_FEEDBACK.kind), так что выбор пользователя.
|
// значением ниже (EMPTY_FEEDBACK.kind), так что выбор пользователя.
|
||||||
export const KIND_OPTIONS: ISelectOption[] = [
|
export const KIND_OPTIONS: ISelectOption[] = [
|
||||||
{ value: 'suggestion', label: 'Предложение' },
|
{ value: 'suggestion', label: 'Предложение' },
|
||||||
{ value: 'complaint', label: 'Жалоба' },
|
{ value: 'complaint', label: 'Обращение' },
|
||||||
{ value: 'other', label: 'Другое' }
|
{ value: 'other', label: 'Другое' }
|
||||||
];
|
];
|
||||||
|
|
||||||
@@ -61,7 +61,7 @@ export interface IAppealFieldsConfig {
|
|||||||
|
|
||||||
/**
|
/**
|
||||||
* Состав полей зависит от вида обращения:
|
* Состав полей зависит от вида обращения:
|
||||||
* жалоба — обязательно описание, а предложение — на желание автора;
|
* обращение — обязательно описание, а предложение — на желание автора;
|
||||||
* предложение — только само предложение, «ситуации» ещё нет;
|
* предложение — только само предложение, «ситуации» ещё нет;
|
||||||
* другое — свободное описание, без предложения.
|
* другое — свободное описание, без предложения.
|
||||||
* Одно правило на форму и на валидацию, чтобы они не разошлись.
|
* Одно правило на форму и на валидацию, чтобы они не разошлись.
|
||||||
|
|||||||
@@ -28,7 +28,7 @@ export const TEXTS = {
|
|||||||
|
|
||||||
placeholders: {
|
placeholders: {
|
||||||
company: 'Выберите компанию',
|
company: 'Выберите компанию',
|
||||||
fullName: 'Иванов Иван Иванович',
|
fullName: 'Нурлан Нурланович Нурланов',
|
||||||
position: 'Например: инженер-проектировщик',
|
position: 'Например: инженер-проектировщик',
|
||||||
kindOther: 'Свой вариант',
|
kindOther: 'Свой вариант',
|
||||||
situation: 'Опишите, что произошло: когда, где, кто участвовал',
|
situation: 'Опишите, что произошло: когда, где, кто участвовал',
|
||||||
@@ -46,7 +46,8 @@ export const TEXTS = {
|
|||||||
required: 'Заполните поле',
|
required: 'Заполните поле',
|
||||||
selectRequired: 'Выберите значение',
|
selectRequired: 'Выберите значение',
|
||||||
tooShort: 'Слишком короткий ответ — опишите подробнее',
|
tooShort: 'Слишком короткий ответ — опишите подробнее',
|
||||||
hasErrors: 'Проверьте выделенные поля — что-то заполнено не полностью.'
|
hasErrors: 'Проверьте выделенные поля — что-то заполнено не полностью.',
|
||||||
|
submitFailed: 'Не удалось отправить обращение. Попробуйте ещё раз позже.'
|
||||||
},
|
},
|
||||||
|
|
||||||
thanks: {
|
thanks: {
|
||||||
|
|||||||
@@ -18,8 +18,8 @@ export interface IFeedbackForm {
|
|||||||
|
|
||||||
/**
|
/**
|
||||||
* Состояние формы обращения: данные, валидация, отправка.
|
* Состояние формы обращения: данные, валидация, отправка.
|
||||||
* Бэкенда пока нет — submit только имитирует отправку (см. TODO ниже),
|
* Отправка идёт через nginx (/api/... -> hr-appeals-auth) — сервис сам
|
||||||
* поэтому вся логика синхронная и живёт здесь, без сервисного слоя.
|
* меняет её на app-only токен и создаёт элемент в списке SharePoint.
|
||||||
*/
|
*/
|
||||||
export function useFeedbackForm(): IFeedbackForm {
|
export function useFeedbackForm(): IFeedbackForm {
|
||||||
const [data, setData] = useState<IFeedbackData>(EMPTY_FEEDBACK);
|
const [data, setData] = useState<IFeedbackData>(EMPTY_FEEDBACK);
|
||||||
@@ -53,11 +53,24 @@ export function useFeedbackForm(): IFeedbackForm {
|
|||||||
setFormError('');
|
setFormError('');
|
||||||
setStatus('submitting');
|
setStatus('submitting');
|
||||||
|
|
||||||
// TODO: заменить на реальную отправку, когда определимся с бэкендом.
|
fetch('/api/feedback', {
|
||||||
// Сейчас это заглушка по договорённости: делаем только UI.
|
method: 'POST',
|
||||||
// eslint-disable-next-line no-console
|
headers: { 'Content-Type': 'application/json' },
|
||||||
console.info('Обращение (заглушка отправки):', data);
|
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');
|
setStatus('done');
|
||||||
|
})
|
||||||
|
.catch((error) => {
|
||||||
|
// eslint-disable-next-line no-console
|
||||||
|
console.error('Не удалось отправить обращение:', error);
|
||||||
|
setFormError(TEXTS.errors.submitFailed);
|
||||||
|
setStatus('idle');
|
||||||
|
});
|
||||||
}, [data]);
|
}, [data]);
|
||||||
|
|
||||||
const reset = useCallback((): void => {
|
const reset = useCallback((): void => {
|
||||||
|
|||||||
@@ -6,7 +6,7 @@ const VALID_COMPANY = '9de064a4-44c8-88ef-11ec-8285925d83d2';
|
|||||||
const LONG_TEXT = 'Развёрнутое описание, которого достаточно для проверки правила';
|
const LONG_TEXT = 'Развёрнутое описание, которого достаточно для проверки правила';
|
||||||
const SHORT_TEXT = 'коротко';
|
const SHORT_TEXT = 'коротко';
|
||||||
|
|
||||||
/** Полностью заполненное «жалоба»-обращение — стартовая точка для тестов. */
|
/** Полностью заполненное обращение — стартовая точка для тестов. */
|
||||||
function buildValidComplaint(overrides: Partial<IFeedbackData> = {}): IFeedbackData {
|
function buildValidComplaint(overrides: Partial<IFeedbackData> = {}): IFeedbackData {
|
||||||
return {
|
return {
|
||||||
...EMPTY_FEEDBACK,
|
...EMPTY_FEEDBACK,
|
||||||
@@ -61,17 +61,17 @@ describe('validateFeedback: вид обращения "Другое"', () => {
|
|||||||
// Состав обязательных полей зависит от вида обращения (см. getAppealFieldsConfig
|
// Состав обязательных полей зависит от вида обращения (см. getAppealFieldsConfig
|
||||||
// в model.ts) — эти тесты фиксируют контракт, чтобы форма и валидация не разошлись.
|
// в model.ts) — эти тесты фиксируют контракт, чтобы форма и валидация не разошлись.
|
||||||
describe('validateFeedback: обязательность полей по видам обращения', () => {
|
describe('validateFeedback: обязательность полей по видам обращения', () => {
|
||||||
it('жалоба: описание ситуации обязательно', () => {
|
it('обращение: описание ситуации обязательно', () => {
|
||||||
const errors = validateFeedback(buildValidComplaint({ situation: '' }));
|
const errors = validateFeedback(buildValidComplaint({ situation: '' }));
|
||||||
expect(errors.situation).toBeDefined();
|
expect(errors.situation).toBeDefined();
|
||||||
});
|
});
|
||||||
|
|
||||||
it('жалоба: предложение по улучшению НЕ обязательно', () => {
|
it('обращение: предложение по улучшению НЕ обязательно', () => {
|
||||||
const errors = validateFeedback(buildValidComplaint({ suggestion: '' }));
|
const errors = validateFeedback(buildValidComplaint({ suggestion: '' }));
|
||||||
expect(errors.suggestion).toBeUndefined();
|
expect(errors.suggestion).toBeUndefined();
|
||||||
});
|
});
|
||||||
|
|
||||||
it('жалоба: если предложение всё же указано, но слишком коротко — ошибка', () => {
|
it('обращение: если предложение всё же указано, но слишком коротко — ошибка', () => {
|
||||||
const errors = validateFeedback(buildValidComplaint({ suggestion: SHORT_TEXT }));
|
const errors = validateFeedback(buildValidComplaint({ suggestion: SHORT_TEXT }));
|
||||||
expect(errors.suggestion).toBeDefined();
|
expect(errors.suggestion).toBeDefined();
|
||||||
});
|
});
|
||||||
|
|||||||
+10
-1
@@ -12,7 +12,16 @@ export default defineConfig({
|
|||||||
},
|
},
|
||||||
server: {
|
server: {
|
||||||
port: 3000,
|
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: {
|
test: {
|
||||||
// globals: true — describe/it/expect доступны без импорта, как привычно по Jest.
|
// globals: true — describe/it/expect доступны без импорта, как привычно по Jest.
|
||||||
|
|||||||
Reference in New Issue
Block a user