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.

Sign up for GitHub

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

Jump to bottom

Add: New keys for departments #705

Merged
merged 23 commits into from
Nov 20, 2023
Merged

Add: New keys for departments #705

merged 23 commits into from
Nov 20, 2023

Conversation

Kar4es
Copy link

@Kar4es Kar4es commented Nov 15, 2023 •
edited
Loading

Что этот PR делает

Добавляет ключики к отделам.

Почему это хорошо для игры

интересный способ попасть в отдел, больше работы для ГП.

Изображения изменений

image

Тестирование

Проверил ключики. Имеют доступ куда надо.

Changelog

🆑
add: Теперь у ГП есть ключи для отделов.
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Nov 15, 2023
@AyIong
Copy link
Collaborator

AyIong commented Nov 15, 2023

Как ты тестил если даже key.dm не заинклюдил в модуль?

@Kar4es
Copy link
Author

Kar4es commented Nov 15, 2023

Как ты тестил если даже key.dm не заинклюдил в модуль?

блять я его удалил случайно

AyIong
AyIong reviewed Nov 15, 2023
View reviewed changes
modular_ss220/objects/code/key.dm Show resolved Hide resolved
modular_ss220/objects/code/key.dm Show resolved Hide resolved
modular_ss220/objects/code/key.dm Show resolved Hide resolved
modular_ss220/objects/code/key.dm Show resolved Hide resolved
modular_ss220/objects/code/key.dm Show resolved Hide resolved
modular_ss220/objects/code/key.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/key.dm Show resolved Hide resolved
modular_ss220/objects/code/key.dm Show resolved Hide resolved
modular_ss220/objects/code/key.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/key.dm Outdated Show resolved Hide resolved
Gaxeer
Gaxeer requested changes Nov 15, 2023
View reviewed changes
modular_ss220/objects/code/key.dm Outdated Show resolved Hide resolved
Gaxeer
Gaxeer reviewed Nov 15, 2023
View reviewed changes
Copy link
Collaborator

@Gaxeer Gaxeer left a comment

Choose a reason for hiding this comment

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

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

@github-actions github-actions bot added 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) labels Nov 15, 2023
@Kar4es Kar4es marked this pull request as ready for review November 15, 2023 10:36
dj-34
dj-34 requested changes Nov 15, 2023
View reviewed changes
modular_ss220/objects/code/key.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/key.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/key.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/key.dm Outdated Show resolved Hide resolved
@PhantornRU PhantornRU added the :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги. label Nov 16, 2023
@Kar4es Kar4es requested a review from dj-34 November 16, 2023 12:24
AyIong
AyIong approved these changes Nov 17, 2023
View reviewed changes
Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

Вроде всё ок

@AyIong AyIong added the 📃 Требуется изменение WIKI Эти изменения необходимо внести на нашу WIKI. label Nov 17, 2023
@Kar4es Kar4es requested a review from Gaxeer November 17, 2023 17:49
dj-34
dj-34 requested changes Nov 18, 2023
View reviewed changes
Copy link
Collaborator

@dj-34 dj-34 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

  1. Че за грязь у спрайтов по бокам?
    image

  2. Разве door_remote не будет открывать двери дистанционно?

@Kar4es
Copy link
Author

Kar4es commented Nov 18, 2023

  1. Че за грязь у спрайтов по бокам?
    image
  2. Разве door_remote не будет открывать двери дистанционно?

я убрал дистанционную активацию. Если грязь у спрайта синего ключа - он будет потом исправлен. Это можно сказать заготовка на будущее, я его все равно поменять планирую

@dj-34
Copy link
Collaborator

dj-34 commented Nov 18, 2023

  1. Че за грязь у спрайтов по бокам?
    image
  2. Разве door_remote не будет открывать двери дистанционно?

я убрал дистанционную активацию. Если грязь у спрайта синего ключа - он будет потом исправлен. Это можно сказать заготовка на будущее, я его все равно поменять планирую

убери грязь сейчас, моветон

@AyIong
Copy link
Collaborator

AyIong commented Nov 20, 2023

4 галки, думаю можно лить

@AyIong AyIong merged commit e196702 into ss220club:master Nov 20, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AyIong AyIong AyIong approved these changes

@Gaxeer Gaxeer Gaxeer approved these changes

@PhantornRU PhantornRU PhantornRU approved these changes

@dj-34 dj-34 dj-34 approved these changes

Assignees
No one assigned
Labels
📃 Требуется изменение WIKI Эти изменения необходимо внести на нашу WIKI. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги. 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. Керберос Этот ПР изменил что-то на Керберосе (DeltaStation)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Kar4es @AyIong @dj-34 @PhantornRU @Gaxeer @ss220bot