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

Том Корпоративного Закона #2172

Merged
merged 13 commits into from
May 31, 2024

Conversation

pxc1984
Copy link
Contributor

@pxc1984 pxc1984 commented May 28, 2024

Описание PR

Добавил книгу с гиперссылкой на корпоративный закон. Добавил эту книгу в лоадаут "Безделушки", в карман кадету СБ и АВД. Также добавил том корпоративного закона в случайные книги в библиотеку. Эта книга может помочь познакомить новичков с вики и корпоративным законом в целом. В дальнейшем планируется добавить корпоративный закон в кпк, а том оставить. предложка

Медиа

Проверки

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

Изменения

🆑 pacable

  • add: Добавлен внутриигровой том с ссылкой на Корпоративный Закон.

@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 May 28, 2024
Copy link
Member

@Morb0 Morb0 left a comment

Choose a reason for hiding this comment

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

Почему именно Book?)
Этот компонент можно навесить на любой активируемый предмет, поэтому думаю стоит получше подумать над названием. Можешь предложить варианты в комменты.

@pxc1984
Copy link
Contributor Author

pxc1984 commented May 28, 2024

Почему именно Book?) Этот компонент можно навесить на любой активируемый предмет, поэтому думаю стоит получше подумать над названием. Можешь предложить варианты в комменты.

Можно просто HyperLink тогда уж?)

@Morb0
Copy link
Member

Morb0 commented May 28, 2024

Почему именно Book?) Этот компонент можно навесить на любой активируемый предмет, поэтому думаю стоит получше подумать над названием. Можешь предложить варианты в комменты.

Можно просто HyperLink тогда уж?)

В целом, можно

@pxc1984 pxc1984 requested a review from Morb0 May 28, 2024 23:20
@lzk228
Copy link
Collaborator

lzk228 commented May 29, 2024

хуйня, КЗ должен бить в самой игре
лучшая идея, как по мне - гайдбук
и то, это оффам

@lapatison
Copy link
Contributor

Тут я с Лазиком - лучше сделать гайдбуком.
Только оффам это не надо\сделают сами.
Так же надо перенести книгу в папку корвакса и сократить описание в три раза.

@NotSoDana
Copy link
Contributor

Кз сейчас постоянно обновляется, переносить его в гайдбук, особенно учитывая как он реализован на вики, крайне нецелесообразно

@pxc1984 pxc1984 requested a review from Morb0 May 30, 2024 15:07
@lapatison
Copy link
Contributor

book-read-verb
image
бога ради, сократи описание
image

@pxc1984
Copy link
Contributor Author

pxc1984 commented May 31, 2024

Благодарю, оба пункта пофиксил. (Решил не добавлять перевод, так как не смог сформулировать, а просто вырезал открытие на ПКМ)

@Morb0
Copy link
Member

Morb0 commented May 31, 2024

Попробуем, изменение не слишком большое и код отделён.

@Morb0 Morb0 merged commit 81f061b into space-syndicate:master May 31, 2024
13 checks passed
@lzk228
Copy link
Collaborator

lzk228 commented May 31, 2024

но всё же суть внутриигрового предмета в том, чтобы читать это внутри игры

@Morb0
Copy link
Member

Morb0 commented May 31, 2024

но всё же суть внутриигрового предмета в том, чтобы читать это внутри игры

В теории полезно новичкам. И так раньше работало в 13, где открывалась страница с вики, возможно не лучший способ, но посмотрим. КЗ и правда слишком часто меняется, чтобы держать его актуальным в игре.

@NotSoDana
Copy link
Contributor

но всё же суть внутриигрового предмета в том, чтобы читать это внутри игры

Удачи уместить весь нынешний КЗ в гайдбук и регулярно обновлять его, учитывая нынешние тенденции его изменений. Это физически сложно и никому не нужна такая ёбка.

@lzk228
Copy link
Collaborator

lzk228 commented May 31, 2024

но всё же суть внутриигрового предмета в том, чтобы читать это внутри игры

В теории полезно новичкам. И так раньше работало в 13, где открывалась страница с вики, возможно не лучший способ, но посмотрим. КЗ и правда слишком часто меняется, чтобы держать его актуальным в игре.

так дело в том, что сам бьёнд работал на майкрософтовском браузере (или типа того) и открывал окошко прямо в игре, потому для него это было легко, а тут сосака отдельная игра, и ты открываешь отдельно браузер который тоже грузит пекич и потом пекич взрывается

@Morb0
Copy link
Member

Morb0 commented May 31, 2024

но всё же суть внутриигрового предмета в том, чтобы читать это внутри игры

В теории полезно новичкам. И так раньше работало в 13, где открывалась страница с вики, возможно не лучший способ, но посмотрим. КЗ и правда слишком часто меняется, чтобы держать его актуальным в игре.

так дело в том, что сам бьёнд работал на майкрософтовском браузере (или типа того) и открывал окошко прямо в игре, потому для него это было легко, а тут сосака отдельная игра, и ты открываешь отдельно браузер который тоже грузит пекич и потом пекич взрывается

Та любой СБшник и так играет с открытым браузером и КЗ. Или они уже знают всё наизусть?

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.

5 participants