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

BonusForFullyDiscovered #2376

Merged
merged 13 commits into from
Dec 26, 2024

Conversation

21Melkuu
Copy link

@21Melkuu 21Melkuu commented Dec 21, 2024

Описание PR

Перезалив, в прошлый раз насрал

Проверки

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

Изменения

🆑 21Gnomkuu"

  • add: Добавлен бонус за полное изучение артефакта

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" Merge Conflict Some content has conflicts. labels Dec 21, 2024
Copy link

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 21, 2024
@21Melkuu
Copy link
Author

@UrPrice бам бим, списки шмоток, локализация и прочее советую проверить если что-то не так

@Kirus59
Copy link
Collaborator

Kirus59 commented Dec 26, 2024

@21Melkuu веселье убираем, закомментированное тоже подсасывает в ЧЛ
image

@21Melkuu
Copy link
Author

@21Melkuu веселье убираем, закомментированное тоже подсасывает в ЧЛ image

👍

@Kirus59 Kirus59 self-assigned this Dec 26, 2024
@Kirus59
Copy link
Collaborator

Kirus59 commented Dec 26, 2024

@UrPrice глянь по прототипам, которые будут спавнится артами при полном изучении.
А то тут предлагают некоторые синди предметы (Омега мыло, шляпа-хамелеон, миксер ДНК, модуль оружия борга), лазерный пистолет, Тесла-пушку и т.д.

@UrPrice
Copy link
Collaborator

UrPrice commented Dec 26, 2024

В остальном - пойдёт. Учитывая концепцию, а иногда и сложность изучить полностью - то это должно быть интересно и весело. Что касается дополнений. Можно было бы вложить различные T3 технологии в этот список, но думаю мы потом вернёмся к этом, когда будем откатывать PR Зонеса на возможность изучения всех технологий.

@21Melkuu
Copy link
Author

В остальном - пойдёт. Учитывая концепцию, а иногда и сложность изучить полностью - то это должно быть интересно и весело. Что касается дополнений. Можно было бы вложить различные T3 технологии в этот список, но думаю мы потом вернёмся к этом, когда будем откатывать PR Зонеса на возможность изучения всех технологий.

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

@Kirus59 Kirus59 merged commit d47d933 into SerbiaStrong-220:master Dec 26, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants