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

Glintwine #618

Merged
merged 9 commits into from
Jan 19, 2024
Merged

Glintwine #618

merged 9 commits into from
Jan 19, 2024

Conversation

Imleriya
Copy link

@Imleriya Imleriya commented Dec 26, 2023

Описание PR

Добавлен новый напиток Глинтвейн. Рецепт 3 юнитов: по 1 юниту каждого реагента - Вино, Сахар, Апельсиновый сок. Для приготовления требуется нагревание от 335 до 370 Кельвинов.
Спрайт напитка сделан SubL для SS220.

Медиа
icon

Проверки

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

Изменения
🆑

  • add: Добавлен напиток Глинтвейн.

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

github-actions bot commented Dec 26, 2023

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

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

State Old New Status
icon Added

Edit: diff updated after ee556fc

Copy link
Member

@TheArturZh TheArturZh left a comment

Choose a reason for hiding this comment

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

в ПР два одинаковых RSI спрайта - один в папке SS220, другой у оффов. Нужно убрать лишний, оставив только тот, что в папке SS220, и поправить пути в прототипах.

Comment on lines 5 to 7
name: глинтвейн
parent: BaseAlcohol
desc: Глядя на этот напиток, вы чувствуете наступление Рождества.
Copy link
Member

Choose a reason for hiding this comment

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

Вынос описания и имени реагента в файл локализации (.ftl) теперь обязателен, иначе не проходит тесты
В поле name и desc нужно указать ключи локализации, reagent-name-glintwine и reagent-desc-glintwine соответственно
Файл для локов наших реагентов нужно создать по пути Resources/Locale/ru-RU/ss220/reagents/meta/drinks.ftl

@TheArturZh
Copy link
Member

С полем physicalDesc нужно сделать то же самое

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

@Ady4ik Ady4ik left a comment

Choose a reason for hiding this comment

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

Я бы убрал отсылку на Нюрнбег, потому что в поисковке и исторических источниках информация рознится.
Если есть желание сделать отсылку, сделай лучше отсылку на "Планету Немецию" или что-то в таком роде.

Copy link

@Ady4ik Ady4ik left a comment

Choose a reason for hiding this comment

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

Я забираю свои слова, я был не прав

@TheArturZh TheArturZh merged commit fe8b2ca into SerbiaStrong-220:master Jan 19, 2024
18 of 19 checks passed
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.

4 participants