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

Умный холодильник. #175

Merged
merged 2 commits into from
Dec 20, 2024

Conversation

Vonsant
Copy link
Collaborator

@Vonsant Vonsant commented Dec 20, 2024

Описание PR

Умный холодильник теперь работает. Вмещает в себя шприцы, таблетки в баночках, бутылки и реагенты из комплектов пополнения. Имеет поиск по названию.

Почему / Баланс

Он был слишком умным, чтобы стоять на каждой станции и ничего не делать. Мы сделали его немного глупее.

Медиа

smartcringe.mp4

Список изменений

🆑

  • add: Умный холодильник теперь умный холодильник.

@github-actions github-actions bot added S: Needs Review ПРу необходим ревью size/L S: Untriaged Changes: Localization Изменяет локализацию Changes: UI and removed S: Untriaged labels Dec 20, 2024
@CaptainSqrBeard
Copy link

CaptainSqrBeard commented Dec 20, 2024

Тестирование

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

Интерфейс

  • В нижней части окна есть строчка, не имеющая смысл для умного холодильника: "Запрашивайте пополнение в отделе снабжения".
  • Строчка "умный фильтр" в верхней части окна тоже не нужна.
  • Элементам в списке холодильника стоило дать внешний вид кнопок, а не просто текста.

изображение

@CaptainSqrBeard
Copy link

А ещё я заметил щас, что у холодильника не меняется спрайт при отсутствии питания

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 20, 2024

Доработаю, но пока что мне нужна статистика по нагрузке на сервер для сравнения. Пойду графану покурю, откачу если что.

@Vonsant Vonsant merged commit c2693c4 into space-syndicate:master Dec 20, 2024
15 checks passed
@CaptainSqrBeard
Copy link

АААААААААА

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 20, 2024

АААААААААА

Я откачу, не пережёвывай

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 20, 2024

Соберу трейс с пары раундов с людьми ток

Vonsant added a commit that referenced this pull request Dec 20, 2024
Vonsant added a commit that referenced this pull request Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменяет локализацию Changes: UI S: Needs Review ПРу необходим ревью size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants