authorization-token #38

Merged
Ghost merged 5 commits from authorization-token into rc 2025-04-21 16:10:31 +03:00
Collaborator
There is no content yet.
VladislavD was assigned by DmitriyA 2025-04-21 10:33:19 +03:00
Ghost was assigned by DmitriyA 2025-04-21 10:33:19 +03:00
DmitriyA added 4 commits 2025-04-21 10:33:20 +03:00
VladislavD approved these changes 2025-04-21 10:37:26 +03:00
DmitriyA reviewed 2025-04-21 10:48:16 +03:00
DmitriyA left a comment
Poster
Collaborator
  1. Была добавлена аутентификация с помощью JWT токенов
  2. Логика подключения графика по веб сокету была вынесена в отдельный компонент
  3. Внедрен компонент skeleton для графиков
  4. Были добавлены статусы для бокового меню
1. Была добавлена аутентификация с помощью JWT токенов 2. Логика подключения графика по веб сокету была вынесена в отдельный компонент 3. Внедрен компонент skeleton для графиков 4. Были добавлены статусы для бокового меню
DmitriyA changed title from authorization-token to WIP: authorization-token 2025-04-21 11:25:37 +03:00
Ghost requested changes 2025-04-21 11:33:50 +03:00
@ -21,1 +56,4 @@
const handleLogout = async () => {
try {
await fetch('http://192.168.2.39:3000/api/auth/logout', {

почему не в .env?

почему не в .env?
src/App.jsx Outdated
@ -23,3 +95,3 @@
<ThemeProvider theme={theme}>
<CssBaseline />
{!isAuthenticated && showLoginModal ? (
{!authState.isAuthenticated && showLoginModal ? (

какое-то неправильное объявление переменных. authState подразумевает состояние (authState.Authenticated), а не isAuthenticated. Подумай над неймингом

какое-то неправильное объявление переменных. authState подразумевает состояние (authState.Authenticated), а не isAuthenticated. Подумай над неймингом
@ -188,2 +135,4 @@
}, [metricName, selectedRange.value, isSelectingRange, calculateStep, processMetricsData]);
const fetchCustomRangeData = useCallback(async () => {
// Добавляем проверку на валидность дат

комментарий убрать

комментарий убрать
@ -203,3 +157,3 @@
if (response.data?.length) {
// Преобразуем данные перед передачей в processMetricsData
// Добавляем нормализацию timestamp

все комментарии убрать

все комментарии убрать
@ -207,3 +161,2 @@
...item,
timestamp: item.timestamp, // оставляем в секундах - processMetricsData конвертирует
value: item.value.toString()
timestamp: item.timestamp > 1e12 ? item.timestamp : item.timestamp * 1000,

что такое ie12? Может лучше в отдельную переменную вывести? Что значит item.timestamp * 1000? Может тоже в отдельную переменную?

что такое _ie12_? Может лучше в отдельную переменную вывести? Что значит item.timestamp * 1000? Может тоже в отдельную переменную?
@ -230,9 +183,10 @@ const PrometheusChart = ({ metricName }) => {
const selectedValue = event.target.value;
const range = TIME_RANGES.find(r => r.value === parseInt(selectedValue, 10));
// Полный сброс состояния перед загрузкой новых данных

убрать

убрать
@ -245,3 +197,1 @@
socketRef.current?.connect();
}
}, []);
// Ждем завершения обновления состояния перед загрузкой

убрать

убрать
@ -287,4 +216,3 @@
}, [fetchData, fetchCustomRangeData, useCustomRange]);
const interpolateData = useCallback((data, targetPointCount) => {
if (!data || data.length < 2) return data;

если нет data вернуть data? поясни

если нет data вернуть data? поясни
Poster
Collaborator

if (!data)
Проверяет, что data существует не null, не undefined и не false.
Если data нет — функция возвращает data, то есть null/undefined.

|| data.length < 2
или проверяет, что в data меньше 2 элементов (массив пуст или содержит 1 элемент).
Если да — функция возвращает исходный data без изменений.

Интерполяция требует как минимум 2 точки начало и конец,если точек меньше, то интерполировать нечего

if (!data) Проверяет, что data существует не null, не undefined и не false. Если data нет — функция возвращает data, то есть null/undefined. || data.length < 2 или проверяет, что в data меньше 2 элементов (массив пуст или содержит 1 элемент). Если да — функция возвращает исходный data без изменений. Интерполяция требует как минимум 2 точки начало и конец,если точек меньше, то интерполировать нечего
@ -18,3 +18,2 @@
try {
// Отправляем данные на бэкенд
const response = await fetch(`${import.meta.env.VITE_BACK_URL}/api/auth/login`, {
const response = await fetch('http://192.168.2.39:3000/api/auth/login', {

в .env

в .env
@ -0,0 +1,20 @@
export const checkAuth = async () => {
try {
const response = await fetch('http://192.168.2.39:3000/api/auth/check', {

в .env

в .env
Ghost requested changes 2025-04-21 11:58:39 +03:00
@ -77,3 +77,3 @@
// Фоновые цвета
background: {
default: "#6CACE4", // Основной фон приложения
default: "#FFFFFF", // Основной фон приложения

BACKGROUND_COLOR = #FFFFFF

BACKGROUND_COLOR = #FFFFFF
DmitriyA added 1 commit 2025-04-21 16:05:34 +03:00
test-org/trust-module-frontend/pipeline/pr-rc This commit looks good Details
b4d653f3a6
corrections after the code review
Ghost approved these changes 2025-04-21 16:10:09 +03:00
Ghost changed title from WIP: authorization-token to authorization-token 2025-04-21 16:10:20 +03:00
Ghost merged commit fd53b187d5 into rc 2025-04-21 16:10:31 +03:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: deployer3000/trust-module-frontend#38
There is no content yet.