Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix admin log filter #572

Closed
wants to merge 1 commit into from
Closed

Conversation

Werzet
Copy link
Collaborator

@Werzet Werzet commented Dec 7, 2023

Описание PR

Сейчас логи пытаются фильтроваться по новой методике HashCod'a. Однако в логи пишутся идентификаторы сущности. Из-за этого админ логи правильно не фильтруются.
Поправил данное поведение возвращением фильтрации по Id.

Медиа

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑

  • fix: фильтрация админ-логов

@TheArturZh
Copy link
Member

space-wizards#22231
может оффы замержат

@DexlerXD
Copy link

DexlerXD commented Dec 8, 2023

ждем мержа оффов?

@Werzet
Copy link
Collaborator Author

Werzet commented Dec 10, 2023

Офы приняли, так что возьмём от них.

@Werzet Werzet closed this Dec 10, 2023
@Werzet Werzet deleted the fix-logs-filter branch December 10, 2023 09:15
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.

3 participants