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

Rufood #738

Merged
merged 52 commits into from
Mar 18, 2024
Merged

Rufood #738

merged 52 commits into from
Mar 18, 2024

Conversation

21Melkuu
Copy link

@21Melkuu 21Melkuu commented Feb 15, 2024

Описание PR

https://discord.com/channels/1097181193939730453/1205125239424557056/1205125239424557056 добавление русской кухни. Там же и вся медиа.

Проверки

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

Изменения

Добавлены новые блюда и напитки, а так же их рецпеты. Список:
✔️Борщ
Миска+ Сырая котлета+ Картофель+ Ломтик лук+ Свекла+ Вода [10]
✔️Щи
Миска+ Сырая котлета+ Картофель+ Ломтик лука+ Капуста+ Вода [10]
Микроволновка 10 сек
✔️Окрошка
Миска+ Мясо курицы+ Картофель+ Ломтик лука+ Вареное яйцо
Микроволновка 10 сек
✔️Окрошка на квасе
Миска+ Мясо курицы+ Картофель+ Ломтик лука+ Вареное яйцо + квас [10]
Микроволновка 10 сек
✔️Винегрет
Миска+ Картофель+ Ломтик лука+ Свекла+ Горох+ Морковь
Микроволновка 10 сек
✔️Чебурек
Мясо+ Тесто+ Соль+ Ломтик лука
Микроволновка 15 сек
✔️Шаурма
Тесто+ Мясо курицы+ Майонез+ Капуста+ Помидор+ Ломтик сыра
Микроволновка 15 сек
✔️Яблочный компот
Яблочный сок + вода
✔️Ягодный компот
Ягодный сок + вода

✔️Добавлены семена свеклы

🆑 21Melkuu

  • add: Добавлены новые блюда, напитки
  • add: Добавлена свекла для выращивания

@github-actions github-actions bot added Changes: Sprites Изменение затронуло файлы ".rsu / .png" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Feb 15, 2024
Copy link

github-actions bot commented Feb 15, 2024

RSI Diff Bot; head commit ea837c0 merging into 52e6983
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Objects/Consumable/Drinks/glass_apple.rsi

State Old New Status
icon Added

Resources/Textures/SS220/Objects/Consumable/Drinks/glass_berry.rsi

State Old New Status
icon Added

Resources/Textures/SS220/Objects/Consumable/Food/bowl.rsi

State Old New Status
borsch Added
bowl Added
okroshka Added
okroshkaKvass Added
shchi Added
vinaigrette Added

Resources/Textures/SS220/Objects/Consumable/Food/cheburek.rsi

State Old New Status
cheburek Added

Resources/Textures/SS220/Objects/Consumable/Food/shawarma.rsi

State Old New Status
shawarma Added
shawarmaGrill Added

Resources/Textures/SS220/Objects/Specific/Hydroponics/beet.rsi

State Old New Status
dead Added
harvest Added
produce Added
seed Added
stage-1 Added
stage-2 Added
stage-3 Added
stage-4 Added
stage-5 Added
stage-6 Added

Edit: diff updated after ea837c0

@21Melkuu
Copy link
Author

опять поднимать...

@21Melkuu
Copy link
Author

Друзья, если даже сейчас я подниму чеки, не мерджите, пока что. И если у вас есть творческие идеи для описания чего-либо, напишите пожалуйста
Я бы въебал ду нот мердж, но не умею

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Feb 15, 2024
@21Melkuu
Copy link
Author

Чеки я поднял, тот что не прошел, не мое. Ждем мистера адуча, в таком случае

@Kit0vras
Copy link

@21Melkuu

@WhitlyOfficial
Copy link

невидимые пиксили убрали, все остальное личное субъективное мнение, не имеющие ничего общего к ревью. UPD: нет, наебал, не убрали. 5 минут и залью еще раз

жду выполнения пунктов ревью, иначе так и будете без апрува сидеть
про ревью
Reviews allow collaborators to comment on the changes proposed in pull requests, approve the changes, or request further changes before the pull request is merged. Repository administrators can require that all pull requests are approved before being merged.

Copy link

@WhitlyOfficial WhitlyOfficial left a comment

Choose a reason for hiding this comment

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

Фикс копирайтов
1213076802873065473

@@ -0,0 +1,119 @@
- type: entity
name: борщ
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.

Потому что если есть старый - новый не нужен, поменяй старый, потому что в новом нет смысла.

Copy link

@WhitlyOfficial WhitlyOfficial left a comment

Choose a reason for hiding this comment

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

Пофиксить бы.

@DexlerXD
Copy link

DexlerXD commented Mar 9, 2024

Попрошу заметить, что отказ в исполнении просьб ревьюверов - вполне обоснованное основание для закрытия ПРа.

@21Melkuu
Copy link
Author

21Melkuu commented Mar 9, 2024

Попрошу заметить, что отказ в исполнении просьб ревьюверов - вполне обоснованное основание для закрытия ПРа.

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

@Kit0vras
Copy link

Kit0vras commented Mar 9, 2024

Попрошу заметить, что отказ в исполнении просьб ревьюверов - вполне обоснованное основание для закрытия ПРа.

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

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

@DexlerXD
Copy link

DexlerXD commented Mar 9, 2024

@21Melkuu менять файлы оффов с нашими пометками - давняя практика. Добавив рецепт и поменяв ссылку на спрайты ты бы ничего не сломал

Copy link

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

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Mar 16, 2024
@Kit0vras
Copy link

Пофикси конфликт и можно принимать

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Mar 16, 2024
@Ady4ik
Copy link

Ady4ik commented Mar 16, 2024

@Kit0vras ласт ревью?

@Ady4ik
Copy link

Ady4ik commented Mar 17, 2024

@TheArturZh merge that

@TheArturZh TheArturZh merged commit 5a116a3 into SerbiaStrong-220:master Mar 18, 2024
16 checks passed
@21Melkuu 21Melkuu deleted the Rufood branch October 13, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants