-
Notifications
You must be signed in to change notification settings - Fork 476
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
base: master
Are you sure you want to change the base?
New station goals #1357
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
цели хуйня
музей, мероприятия, мотивация персонала, это не очень похоже на то, чем будет заниматься научная станция
цель на колапс аномалий так вообще узаконенный гриф
Обращаем Ваше внимание, что распространение содержания данного документа среди лиц, не обладающих юридической неприкосновенностью, строго запрещено из-за возможности дискредитации руководства Корпорации. Поэтому для представления новой цели командованию капитану необходимо обратиться к нему с устным докладом. | ||
═════════════════════════════════════════ | ||
Место для печатей | ||
station-goal-artifacts = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
я бы сменил на
station-goal-artifacts = | |
station-goal-anomalies = |
ну и айди прототипа тоже надо сменить
Цели - класс |
Чуствуется ирония. Ну а вообще да, слабоваты цели. Может для новых целей использовать новые механики? |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
@ficcialfaint Когда закроешь? Тут уже успели #1461 мержнуть |
@ficcialfaint are you coming back to this? |
dead pr xd |
Цель на ксенобиология есть, но в этом пре более подробно расписано, надо ещё цели на аванпост и теслу раписать, а так, цели прикольные, некоторые рассчитаны на рп и почти никак на смену не влияют, как та же цель на награждение лучших сотрудников, некоторые на креативность, как когда капитан сам должен назначить цель, но вот цель на повышение боеспособности будет антагов душить с вечными обысками, хотя думаю что прикольно |
@ficcialfaint gonflict |
Описание PR
Шаблон целей станции был обновлён, а вместе с ним обновлены старые и добавлены новые цели станции. Текст написан @cap_of_tea
Как система получает код станции?
Дело в том, что легкого пути получения ID/код станции нет (но могу заПРить визардам), поэтому я использовал следующее Regex-выражение →
.*-(\d+)$
. Проверить регулярку можно тут. Если станция не имеет кода (например как Dev станция), и не найдено никаких совподений по выражению, то выводятся???
(можно увидеть в скриншотах ниже).Какая выводится дата?
Дата выводится текущая для сервера, но с увеличенным годом на тысячу (например, вместо 2023 г. будет 3023 г.). Выводится все в формате
dd.MM.yyyy
(например,22.08.3023
).Медиа
Здесь у станции Dev нет кода (в чате можно заметить, что даже там выводится просто название карты,
Dev
), поэтому стоят вопросики. А также не смотрите на указанную дату, ведь изначально я прибавлял не тысячу лет, а всего лишь один годВ правом верхнем углу можно увидеть как в чате, так и в КПК название станции вместе с её кодом. То же самое число написано на листочке
Проверки
Изменения
🆑 faint