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

IC контрабанда и знания персонала об предметах #1767

Merged
merged 139 commits into from
Mar 20, 2024

Conversation

TheShuEd
Copy link
Contributor

@TheShuEd TheShuEd commented Jan 15, 2024

Описание PR

Мы берем #1731
и дополняем общую неметовую информацию новыми пометками, которые показываются только для определенных ролей или антагов.
Вот так:

2024-01-16.16-40-14.mp4

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

По умолчанию гиппоручка будет описана как обычная ручка. Химики голубым цветом увидят пометку "В эту ручку встроен замаскированный гипоспрей", Антагонисты увидят кроваво-красную надпись "Это орудие химического убийства, способное незаметно впрыскивать в цель снотворное или яды, разработанное синдикатом"

Первый плюс: Увеличение погружения и атмосферы рп.
Разные роли в раунде будут давать разные описания, как бы приоткрывая информацию о предмете с разных сторон. Так же дает психологическую важность самой роли через игромехан.

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

Согласие от морти получено: https://discord.com/channels/919301044784226385/924735650119422032/1196584720994148494

Времять работац

Проверки

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

Изменения
🆑 TheShuEd, AlexandrSTG

  • add: Теперь антагонисты и игроки определенных ролей при просмотре некоторых предметов могут видеть дополнительную информацию, опираясь на страницу Контрабанды из вики.
  • remove: Убраны все упоминания синдиката и ООС описания предметов антагонистов.

@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation labels Jan 15, 2024
@TheShuEd TheShuEd marked this pull request as draft January 15, 2024 23:03
@TheShuEd
Copy link
Contributor Author

2024-01-16.16-40-14.mp4

Обновил видос в описании

@TheShuEd TheShuEd marked this pull request as ready for review January 16, 2024 20:31
@TheShuEd
Copy link
Contributor Author

@ficcialfaint @JerryImMouse заправшиваю ревью по орфографии

@TheShuEd
Copy link
Contributor Author

ПР готов. Возможны незамеченные косяки, так как много муторной работы. Но я перенес всю страницу контрабанды в IC-пространство
https://station14.ru/wiki/%D0%9A%D0%BE%D0%BD%D1%82%D1%80%D0%B0%D0%B1%D0%B0%D0%BD%D0%B4%D0%B0

@TheShuEd TheShuEd changed the title Избавляемся от меты, часть вторая IC контрабанда и знания персонала об предметах Jan 16, 2024
@TheShuEd
Copy link
Contributor Author

2024-01-16.23-42-21.mp4

Немного примеров работы с имплантами.

@TheShuEd TheShuEd requested a review from Morb0 March 20, 2024 18:33
@TheShuEd
Copy link
Contributor Author

TheShuEd commented Mar 20, 2024

выдохнули, ПР завершен. Если Лазик еще раз решит сломать мне все, я вычислю его айпи адрес, приду домой и задушу

@Morb0
Copy link
Member

Morb0 commented Mar 20, 2024

Let's try

@Morb0 Morb0 merged commit cec88ae into space-syndicate:master Mar 20, 2024
10 checks passed
@lzk228
Copy link
Collaborator

lzk228 commented Mar 21, 2024

НЕЕЕЕЕ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants