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

Add support for min and max player for station goal #2301

Merged
merged 2 commits into from
Jun 19, 2024

Conversation

lzk228
Copy link
Collaborator

@lzk228 lzk228 commented Jun 19, 2024

Описание PR

Добавил возможность ограничить цели по количеству игроков на сервере
Чтобы 10 тел на станции не строили теслу 4 часа и чтобы 10 инженеров на дельте не построили зоопарк за 20 минут.

Проверки

Протестировал локально, поставив на все кроме одной цели ограничение в мин 3 игрока, выпадала только доступная цель

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

@lzk228 lzk228 requested a review from Morb0 as a code owner June 19, 2024 13:59
@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged label Jun 19, 2024
@NotSoDana
Copy link
Contributor

Ещё б новых целей завезли, а то какой-нибудь Мейн умрёт каждую смену теслу/сингу строить

@UmnayaEbanushka
Copy link

Мейн умрёт

Слава богу

@p0etus
Copy link

p0etus commented Jun 19, 2024

Ещё б новых целей завезли

Канал для новых целей станции всё-ещё не удалили. Морти должен был цели посмотреть с него, и выбрать оттуда крутые. Ждём :) (Чёт часто стал это слово использовать. XD)

@lzk228
Copy link
Collaborator Author

lzk228 commented Jun 19, 2024

Морти

:trollface:

@lzk228 lzk228 changed the title Add min and max player to station goal Add support for min and max player to station goal Jun 19, 2024
@lzk228 lzk228 changed the title Add support for min and max player to station goal Add support for min and max player for station goal Jun 19, 2024
@NotSoDana
Copy link
Contributor

Канал для новых целей станции всё-ещё не удалили. Морти должен был цели посмотреть с него, и выбрать оттуда крутые. Ждём :) (Чёт часто стал это слово использовать. XD)

Насколько я помню, этим должен был заняться Алекс, но его ближайшие полгода точно не будет дэмн. Такое ощущение, что будет проще самому предложку сделать куда собрать идей от игроков и залить их, ибо доступа к чату нет

@Morb0 Morb0 merged commit d01c1cc into space-syndicate:master Jun 19, 2024
9 of 10 checks passed
@lzk228 lzk228 deleted the 06-19-station-goal-update branch June 19, 2024 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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