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

Extractable items for Ghetto Chemistry #209

Merged

Conversation

DjfjdfofdjfjD
Copy link

@DjfjdfofdjfjD DjfjdfofdjfjD commented Jul 31, 2023

О запросе слияния

Допиленная версия #207, вроде как рабочая. Новым ПРом потому что я с бранчами напортачил люто
Добавляет возможность измельчать некоторые вещи, чего достаточно для варки этих наркотиков без доступа в химию:

  1. Метафетамин
  2. Импедризин
  3. Космические Наркотики
    И другие препараты...

Медиа

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

Чейнджлог

🆑

  • tweak: В измельчителе теперь можно выжимать или измельчать некоторые предметы, чего достаточно для создания некоторых наркотиков и препаратов в "домашних" условиях.

@github-actions github-actions bot added the No C# label Jul 31, 2023
solutions:
food:
reagents:
- ReagentId: Sugar
Copy link
Collaborator

Choose a reason for hiding this comment

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

5 Сахара заменяет 10 унаследованных Нутриентов. Да и не только сахар в батончике. Сахар сейчас не утоляет голод, так что добавляй ещё 10 нутриентов. К тому же ты убрал максимальный объём, что у парента в 30. Так что чини.

Copy link
Author

Choose a reason for hiding this comment

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

5 Сахара заменяет 10 унаследованных Нутриентов. Да и не только сахар в батончике. Сахар сейчас не утоляет голод, так что добавляй ещё 10 нутриентов. К тому же ты убрал максимальный объём, что у парента в 30. Так что чини.

поправлю это

solutions:
pen:
reagents:
- ReagentId: Iodine
Copy link
Collaborator

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.

Почему ручки буду состоять из Йода?

Условно чернила, взято с ТГ, тоже для мета

solutions:
food:
reagents:
- ReagentId: Sugar
Copy link
Collaborator

Choose a reason for hiding this comment

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

Тут тоже самое

@@ -276,10 +291,13 @@
- type: SolutionContainerManager
solutions:
food:
maxVol: 1
Copy link
Collaborator

Choose a reason for hiding this comment

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

Верни ограничение ёмкости.

solutions:
bulb:
reagents:
- ReagentId: Nitrogen
Copy link
Collaborator

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.

В лампочках нет Азота, это миф. В прошлом был, но от этого быстро отказались, в связи с быстрой изнашиваемостью. Так что там Вакуум.

Это взято с ТГ и нужно для расширения возможностей гетто химии, без этого Мет не сделать

Quantity: 10
- ReagentId: Hydrogen
Quantity: 10
- type: Extractable
Copy link
Collaborator

Choose a reason for hiding this comment

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

У тебя съехал Тип влево, из-за этого ошибки.

@DjfjdfofdjfjD
Copy link
Author

DjfjdfofdjfjD commented Aug 1, 2023

you must merge my pr
now

@MilenVolf
Copy link
Collaborator

MilenVolf commented Aug 1, 2023

Наверное стоит переименовать на "Extractable items" или что-то в этом роде. "Chemistry V3" звучит так, будто была перелопачена сама Химия. Название ПР-а должно говорить, что он изменил.

@DjfjdfofdjfjD
Copy link
Author

Наверное стоит переименовать на "Extractable items" или что-то в этом роде. "Chemistry V3" звучит так, будто была перелопачена сама Химия. Название ПР-а должно говорить, что он изменил.

не думаю, этот пр называется в честь гайда на вики ТГ "Guide to Ghetto Chemistry", благодаря которому эта идея и появилась

@MilenVolf
Copy link
Collaborator

MilenVolf commented Aug 1, 2023

Наверное стоит переименовать на "Extractable items" или что-то в этом роде. "Chemistry V3" звучит так, будто была перелопачена сама Химия. Название ПР-а должно говорить, что он изменил.

не думаю, этот пр называется в честь гайда на вики ТГ "Guide to Ghetto Chemistry", благодаря которому эта идея и появилась

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

@MilenVolf
Copy link
Collaborator

Допустим: "Extractable items for Ghetto Chemistry". Так пойдёт

@DjfjdfofdjfjD DjfjdfofdjfjD changed the title Ghetto surg... Chemistry V3 Extractable items for Ghetto Chemistry Aug 1, 2023
@MilenVolf
Copy link
Collaborator

В ченджлоге поменяй слово "новые" на "некоторые"

Copy link

@Mr0maks Mr0maks left a comment

Choose a reason for hiding this comment

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

Теперь СБ будет чем заняться. LGTM.

@Mr0maks Mr0maks merged commit da07d2c into Workbench-Team:arumoon-server Aug 1, 2023
15 checks passed
@DjfjdfofdjfjD DjfjdfofdjfjD deleted the ghetto-chemistry-v3 branch August 18, 2023 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants