Skip to content

upgrade logs page#735

Open
ninedev-i wants to merge 12 commits intomasterfrom
feature/logs-page
Open

upgrade logs page#735
ninedev-i wants to merge 12 commits intomasterfrom
feature/logs-page

Conversation

@ninedev-i
Copy link
Copy Markdown
Contributor

Переделал страницу логов. Исправил баги с копированием. Изменил дефолтное направление скролла. Убрал дублирующие логи при подгрузке.
image

@ninedev-i ninedev-i requested a review from KraPete April 24, 2025 13:30
@ninedev-i ninedev-i requested a review from a team as a code owner April 24, 2025 13:30
@ekateluv
Copy link
Copy Markdown
Contributor

Что то у меня не взлетело. Открываю системный журнал, на секунду мелькает интерфейс логов и потом белый экран
image

@ekateluv
Copy link
Copy Markdown
Contributor

А мы разве не чинили дублирование логов при подгрузке?

@sikmir
Copy link
Copy Markdown
Contributor

sikmir commented Apr 29, 2025

А мы разве не чинили дублирование логов при подгрузке?

Чинили, причем на стороне wb-mqtt-logs.

@ninedev-i
Copy link
Copy Markdown
Contributor Author

ninedev-i commented Apr 30, 2025

Что то у меня не взлетело. Открываю системный журнал, на секунду мелькает интерфейс логов и потом белый экран

@ekateluv а ты установила новые зависимости? из папки frontend npm install

@ekateluv
Copy link
Copy Markdown
Contributor

а ты установила новые зависимости

просто собранные пакеты из дженкиса установила, то что в твоей ветке собралось

@ninedev-i
Copy link
Copy Markdown
Contributor Author

@ekateluv починил ошибку с пустой страницей

@ekateluv
Copy link
Copy Markdown
Contributor

не помогает :( тестинг
изображение

# Conflicts:
#	debian/changelog
#	frontend/app/scripts/app.js
#	frontend/app/scripts/controllers/logsController.js
#	frontend/app/views/logs.html
#	frontend/src/components/dropdown/dropdown.tsx
#	frontend/src/components/dropdown/index.ts
#	frontend/src/components/dropdown/styles.css
#	frontend/src/components/dropdown/types.ts
#	frontend/src/components/input/styles.css
@ninedev-i
Copy link
Copy Markdown
Contributor Author

не помогает :( тестинг изображение

нашел в чем там было дело и поправил

@ekateluv
Copy link
Copy Markdown
Contributor

ekateluv commented Oct 2, 2025

Запись экрана от 2025-10-02 13-12-00.webm
Скроллю вверх и меня сильно отбрасывает куда то в зад

Copy link
Copy Markdown
Contributor

@KraPete KraPete left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Основная проблема меня как пользователя - нет индикатора того, что выполняется запрос. Я что-то ввёл или выбрал, ничего не меняется, я начинаю дёргаться.

  2. Размер фильтра скачет при вводе, в результате в списке элементы по две строки.Image

  3. Места полно, а элемент списка опять в две строки.Image

  4. Ввожу в фильтре re{, пишет, что нет элементов. Стираю фильтр, всё ещё нет.

Image
  1. Постоянно выпадает ошибка, что сервис недоступен, хотя по логам контроллера вижу, что сервис работает. Надо просто активно менять фильтры и вводить шаблоны для поиска
Image
  1. Не подгружаются новые сообщения в логе. Загружаю страницу. Делаю колесом мышки вверх-вниз, ожидаю, что при достижении ниднего края появятся новые сообщения, а их нет. Жму F5 получаю кучу сообщений.

import { HTMLAttributes } from 'react';
import './styles.css';

export const FormGroup = ({ children }: HTMLAttributes<HTMLDivElement>) => (
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут бы комментарий для чего оно. Есть уже componets/form и там куча компонентов.
Судя по названию, это должно быть там же?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это визуальный компонент, который склеивает вместе инпуты и кнопки

# Conflicts:
#	debian/changelog
#	frontend/app/scripts/i18n/react/locales/en/translations.json
#	frontend/app/scripts/i18n/react/locales/ru/translations.json
#	frontend/app/views/logs.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants