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.

Sign up for GitHub

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

Jump to bottom

Smart fridge #555

Merged
merged 29 commits into from
Dec 10, 2023
Merged

Smart fridge #555

merged 29 commits into from
Dec 10, 2023

Conversation

SkaldetSkaeg
Copy link

@SkaldetSkaeg SkaldetSkaeg commented Dec 3, 2023 •
edited
Loading

Описание PR

Добавлена отдельная модель умного холодильника (SmartFridge [SS220]) учитывающего указанные в задаче функционал:
image
Медиа

image image image

Проверки

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

Изменения

🆑
add: Новый вый вид умных холодильников с более быстрой выдачей, адекватным указанием имен в меню и возможностью перемещать предметы из контейнеров.

@SkaldetSkaeg
Переписываю с 0
b79907f
@SkaldetSkaeg
3 попытки совмещения ужа и ежа, на этот раз полностью по предложению …
743b485 
…Артура
@SkaldetSkaeg
Merge branch 'SmartFridge' of https://github.com/SkaldetSkaeg/space-s…
0b9544d 
…tation-14-220 into SmartFridge
@SkaldetSkaeg
Первые потуги третьей попытки совместить ужа Storage и Dtictionary Ve…
668eb52 
…nding
@SkaldetSkaeg
Адекватное отображение в новом интерфейсе
669fd94
@SkaldetSkaeg
Merge branch 'SerbiaStrong-220:master' into SmartFridge
b4f3cd4
@SkaldetSkaeg
Добработки работы интерфейса с Storage (черновой вариант)
ee8cd24
@SkaldetSkaeg
Merge branch 'SmartFridge' of https://github.com/SkaldetSkaeg/space-s…
850975c 
…tation-14-220 into SmartFridge
@SkaldetSkaeg
Function of getting items adjusted
28b789e
@SkaldetSkaeg
Pre-PR state
7f9a43f
@SkaldetSkaeg
done
4f2491c
@SkaldetSkaeg
Merge branch 'SerbiaStrong-220:master' into SmartFridge
2b219d8
@SkaldetSkaeg
Merge branch 'SerbiaStrong-220:master' into SmartFridge
8afe5a2
@github-actions github-actions bot added the Changes: UI Изменение затронуло файлы ".xaml" label Dec 3, 2023
@Kit0vras
Copy link

Kit0vras commented Dec 3, 2023

Нужно внести холодильник в Resources/Migration.yml, чтобы все умные холодильники на картах заменялись на эти

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Dec 3, 2023
Kit0vras
Kit0vras reviewed Dec 3, 2023
View reviewed changes
Resources/Locale/ru-RU/ss220/smartftidge.ftl
@@ -0,0 +1,2 @@
ent-StorageSmartFridge = Умный холодильник
Copy link

Choose a reason for hiding this comment

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

Suggested change
ent-StorageSmartFridge = Умный холодильник
ent-StorageSmartFridge = умный холодильник

Copy link
Author

Choose a reason for hiding this comment

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

но тут как в оригинале

Copy link

Choose a reason for hiding this comment

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

Я понял почему так, видимо они считают это именем собственным, как название торгового автомата

@Kit0vras
Copy link

Kit0vras commented Dec 3, 2023

а овощи в нём хранить можно? потому что там написано только о семенах
в 13 ботаники в нём хранят овощи

@Kit0vras
Copy link

Kit0vras commented Dec 3, 2023 •
edited
Loading

под строчкой "изменения" поставь 🆑
image

@SkaldetSkaeg
Copy link
Author

а овощи в нём хранить можно? потому что там написано только о семенах в 13 ботаники в нём хранят овощи

Да, как я понял, тут - Produce

@Kit0vras
Copy link

Kit0vras commented Dec 3, 2023

под строчкой "изменения" поставь 🆑 image

не, чтоб прямо было под строчкой, вот так:

Изменения
🆑

  • add: блаблабла

@Kit0vras
Copy link

Kit0vras commented Dec 3, 2023

ник писать не обязательно, если ничего не писать - то используется твой ник гитхаба

@SkaldetSkaeg
Copy link
Author

а овощи в нём хранить можно? потому что там написано только о семенах в 13 ботаники в нём хранят овощи

По итогу, ты меня на интересный баг натолкнул странного срабатывания whitelist для овощей.

@SkaldetSkaeg
Поправлены тэги (очень странно работает со Storage, так как предмет п…
e062627 
…роверяется и по тэгу, и по компонентам, в результате предмет может вставиться, написав ошибку вайтлиста)
@SkaldetSkaeg SkaldetSkaeg requested a review from Kit0vras December 3, 2023 19:40
@Kit0vras
Copy link

Kit0vras commented Dec 4, 2023

Насчёт прототипов косяков не вижу, насчёт кода - ждём Декслера

DexlerXD
DexlerXD suggested changes Dec 4, 2023
View reviewed changes
Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

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

Выглядит круто, но есть вопрос к выдаче таблеток: если он должен их выдавать быстро, то не логично ли будет таблетки выдавать не в руки, а на пол? Скорость работы от быстрой выдачи, если она происходит в руки, не особо увеличивается.

Content.Server/SS220/MachineStorage/EntityMachineStorageSystem.cs Outdated
Copy link

Choose a reason for hiding this comment

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

???

Copy link
Author

Choose a reason for hiding this comment

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

Надо будет такое мне проверять получше. Пережиток старой реализации.

По поводу выдачи на пол. Могу добить эту реализацию, так как начинал.

Copy link

Choose a reason for hiding this comment

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

Надо будет такое мне проверять получше. Пережиток старой реализации.

По поводу выдачи на пол. Могу добить эту реализацию, так как начинал.

Давай, будет круто

@21Melkuu
Copy link

21Melkuu commented Dec 5, 2023

ура наконец холодильники действительно умные, мб это и оффам нунжно?

DexlerXD
DexlerXD suggested changes Dec 5, 2023
View reviewed changes
Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

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

в шапки файлов лицензию добавь еще

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Dec 6, 2023
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 6, 2023

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Dec 10, 2023
DexlerXD
DexlerXD suggested changes Dec 10, 2023
View reviewed changes
Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

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

изображение
исправить бы надо

@SkaldetSkaeg
Copy link
Author

интересно

DexlerXD
DexlerXD suggested changes Dec 10, 2023
View reviewed changes
Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

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

изображение
теперь другое

@DexlerXD DexlerXD merged commit 598e8f6 into SerbiaStrong-220:master Dec 10, 2023
13 checks passed
@SkaldetSkaeg SkaldetSkaeg deleted the SmartFridge branch December 24, 2023 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kit0vras Kit0vras Kit0vras approved these changes

@DexlerXD DexlerXD DexlerXD approved these changes

Assignees
No one assigned
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: UI Изменение затронуло файлы ".xaml" Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@SkaldetSkaeg @Kit0vras @21Melkuu @DexlerXD