redesign #34

Closed
DmitriyA wants to merge 0 commits from redesign into rc
Collaborator
There is no content yet.
VladislavD was assigned by DmitriyA 2025-04-09 17:20:38 +03:00
Ghost was assigned by DmitriyA 2025-04-09 17:20:38 +03:00
DmitriyA added 16 commits 2025-04-09 17:20:39 +03:00
test-org/trust-module-frontend/pipeline/pr-rc This commit looks good Details
bd96278895
redesign and fix graphics
test-org/trust-module-frontend/pipeline/pr-rc This commit looks good Details
4405c693aa
Established a connection to the back using a web socket
test-org/trust-module-frontend/pipeline/pr-rc This commit looks good Details
32ece2f0ff
refactoring, fixed bugs with the web socket
test-org/trust-module-frontend/pipeline/pr-rc This commit looks good Details
a24b89220c
fixed bugs
test-org/trust-module-frontend/pipeline/pr-redesign This commit looks good Details
b6b3b36f5a
fixed data interpolation and range allocation
test-org/trust-module-frontend/pipeline/pr-redesign Build queued... Details
e56b82bb66
optimized chart loading
test-org/trust-module-frontend/pipeline/pr-redesign Build queued... Details
5b25876056
added the api prefix
Ghost requested changes 2025-04-09 17:33:49 +03:00
@ -39,2 +55,2 @@
setSelectionStart(null);
setSelectionEnd(null);
// Если диапазон больше 24 часов - показываем дату
if (range > 86400000) {

время лучше выносит в енумы или отдельные переменные. Читабельность кода плохая. Пример 24_HOURS (но лучше подумать над названием) = 86400000, а затем подставляешь это. Так же вопрос, это в милисекундах или секундах?

время лучше выносит в енумы или отдельные переменные. Читабельность кода плохая. Пример 24_HOURS (но лучше подумать над названием) = 86400000, а затем подставляешь это. Так же вопрос, это в милисекундах или секундах?
@ -6,2 +6,4 @@
const trimmedData = data.slice(-20);
const CustomTooltip = ({ active, payload, label }) => {
if (active && payload && payload.length) {

Немножко странное условие, тебе необходимо, чтобы точно был payload, и чтобы payload.length был не пустой?

Немножко странное условие, тебе необходимо, чтобы точно был payload, и чтобы payload.length был не пустой?
@ -2,3 +2,3 @@
import { Box, styled } from "@mui/material";
import SidebarMenu from "./SidebarMenu";
import "../../Style/Dashboard.css";
import { statusManager1, statusManager2 } from "../TreeChart/dataUtils";

Вообще, насколько я знаю, что можно установить путь app приложения (корень). тогда все пути будут типа app/.../TreeChart/dataUtils, но это на подумать

Вообще, насколько я знаю, что можно установить путь app приложения (корень). тогда все пути будут типа app/.../TreeChart/dataUtils, но это на подумать
VladislavD requested changes 2025-04-10 09:36:08 +03:00
@ -39,1 +25,3 @@
if (data.length >= minPoints) return data;
const formatTime = useCallback((timestamp, rangeSeconds) => {
const ts = typeof timestamp === 'number' ? timestamp : Date.now();
const date = new Date(ts);
Collaborator
+ const ts = typeof timestamp === 'number' ? new Date(timestamp) : Date.now();
- const date = new Date(ts);
``` + const ts = typeof timestamp === 'number' ? new Date(timestamp) : Date.now(); - const date = new Date(ts); ```
DmitriyA added 2 commits 2025-04-10 12:36:50 +03:00
test-org/trust-module-frontend/pipeline/pr-redesign Build queued... Details
22c5fcf02c
Converted time to constants
DmitriyA closed this pull request 2025-04-10 12:37:18 +03:00
All checks were successful
test-org/trust-module-frontend/pipeline/pr-rc This commit looks good

Pull request closed

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#34
There is no content yet.