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

New station goals #1357

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ficcialfaint
Copy link
Collaborator

@ficcialfaint ficcialfaint commented Aug 22, 2023

Описание PR

Шаблон целей станции был обновлён, а вместе с ним обновлены старые и добавлены новые цели станции. Текст написан @cap_of_tea

Как система получает код станции?

Дело в том, что легкого пути получения ID/код станции нет (но могу заПРить визардам), поэтому я использовал следующее Regex-выражение → .*-(\d+)$. Проверить регулярку можно тут. Если станция не имеет кода (например как Dev станция), и не найдено никаких совподений по выражению, то выводятся ??? (можно увидеть в скриншотах ниже).

Какая выводится дата?

Дата выводится текущая для сервера, но с увеличенным годом на тысячу (например, вместо 2023 г. будет 3023 г.). Выводится все в формате dd.MM.yyyy (например, 22.08.3023).

Медиа

Content Client_RPphEgT3Wv
Здесь у станции Dev нет кода (в чате можно заметить, что даже там выводится просто название карты, Dev), поэтому стоят вопросики. А также не смотрите на указанную дату, ведь изначально я прибавлял не тысячу лет, а всего лишь один год

Content Client_a4FCSQm5yT
В правом верхнем углу можно увидеть как в чате, так и в КПК название станции вместе с её кодом. То же самое число написано на листочке

Проверки

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

Изменения

🆑 faint

  • tweak: Nanotrasen ввели поправки в формулировки целей станции, добавили новые цели, а также ввели в действие новую форму NT-№???-CC-CMD-DCR

@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 Aug 22, 2023
Copy link
Contributor

@lzk228 lzk228 left a comment

Choose a reason for hiding this comment

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

цели хуйня
музей, мероприятия, мотивация персонала, это не очень похоже на то, чем будет заниматься научная станция
цель на колапс аномалий так вообще узаконенный гриф

Обращаем Ваше внимание, что распространение содержания данного документа среди лиц, не обладающих юридической неприкосновенностью, строго запрещено из-за возможности дискредитации руководства Корпорации. Поэтому для представления новой цели командованию капитану необходимо обратиться к нему с устным докладом.
═════════════════════════════════════════
Место для печатей
station-goal-artifacts =
Copy link
Contributor

Choose a reason for hiding this comment

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

я бы сменил на

Suggested change
station-goal-artifacts =
station-goal-anomalies =

ну и айди прототипа тоже надо сменить

@lapatison
Copy link
Contributor

Цели - класс

@Nimfar11
Copy link
Contributor

Nimfar11 commented Sep 21, 2023

Цели - класс

Чуствуется ирония.

Ну а вообще да, слабоваты цели. Может для новых целей использовать новые механики?
Придумать цель с теми же боргами. Типа сделать определенное количество боргов и модулей, а после продать (условно продать необходимому покупателю) их, бланки с ценой доставить на ЦК.
Ксенобиология интересна, можно испытания там проводить с помощью мутированных растений. Инженеры строят, ботаники мутируют разные растения, ученые проводят опыты.
Ну и дальше что-нибудь, да хоть новый арсенал на изолирвоанном спутнике.

@comasqw comasqw mentioned this pull request Sep 26, 2023
4 tasks
@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label Dec 8, 2023
Copy link
Contributor

github-actions bot commented Dec 8, 2023

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@JerryImMouse
Copy link
Contributor

@ficcialfaint Когда закроешь? Тут уже успели #1461 мержнуть :trollface:

@lzk228
Copy link
Contributor

lzk228 commented Jan 31, 2024

@ficcialfaint are you coming back to this?

@NikaVichy
Copy link
Contributor

dead pr xd

@SoulFN
Copy link
Contributor

SoulFN commented Mar 31, 2024

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

@lzk228
Copy link
Contributor

lzk228 commented Apr 23, 2024

@ficcialfaint gonflict

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 Merge Conflict PR have unsolved merge conflicts 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.

None yet

7 participants