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: New Station "Nova" #5461

Merged
merged 81 commits into from
Sep 25, 2024
Merged

add: New Station "Nova" #5461

merged 81 commits into from
Sep 25, 2024

Conversation

PiroMage
Copy link
Contributor

Описание

Ссылка на предложение/Причина создания ПР

Демонстрация изменений

@github-actions github-actions bot added the DMI Affects DMI files label Jul 15, 2024
@Den109G
Copy link
Contributor

Den109G commented Jul 15, 2024

Если ПР не готов (станции как я вижу нет) - можешь перевести этот ПР в драфт для доработок. Где-то справа есть кнопка для этого.

@PiroMage PiroMage requested a review from a team as a code owner July 15, 2024 08:28
@github-actions github-actions bot added the MAP Affects map label Jul 15, 2024
@aeternaclose aeternaclose marked this pull request as draft July 15, 2024 09:00
@github-actions github-actions bot added Merge Conflict This doesn't seem right and removed Merge Conflict This doesn't seem right labels Jul 25, 2024
@SAADf603
Copy link
Contributor

SAADf603 commented Jul 30, 2024

Ревью
image

@SAADf603
Copy link
Contributor

Ловушка рефактора:
image
мышь/серая теперь просто мышь
шкаф/барный теперь шкаф/защищённый_шкаф/барный (то же самое должно быть с детективом)

помимо ловушки рефактора:
image
насрано латисами где попало и неверно указан путь зоны строительная/коридор

@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Aug 3, 2024
@github-actions github-actions bot added TGUI Affects TGUI javascript Pull requests that update Javascript code labels Aug 5, 2024
@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Aug 5, 2024
@PiroMage PiroMage marked this pull request as ready for review August 5, 2024 22:45
Daeberdir
Daeberdir previously approved these changes Aug 5, 2024
@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Aug 17, 2024
Убран эффект инициализации, вероятно, с маркеров шахтерских
1) Добавлен переход между вторым этажом механика и техами мостика.
2) Дополнительные выходы в космос в переходе механика.
3) Добавлены решетки, где они должны быть, убраны там, где они выглядят неподходяще.
4) Добавлена комнатка в техи медотдела.
5) Изменена дверь в театре с ботанической на дверь с общим доступом.
@Bizzonium
Copy link
Member

Сделаешь ребейз или мерж новых коммитов? Если тебе будет проще сделать апдейт через мерж, то можно и так, теперь это не так критично.

@Daeberdir
Copy link
Contributor

Всё же стоит дотерпеть до момента, когда я смогу сказать, что мапа готова к админским тестам. Если нужно это приоритезировать - сделаю.

@Bizzonium
Copy link
Member

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

@Daeberdir
Copy link
Contributor

Ошибки Фаррагуса мы повторять не будем. Первое впечатление игроков на релизе именно станционной мапы - очень важно.

PiroMage and others added 10 commits September 18, 2024 03:50
1) Добавлены свои зоны для туалета прибытия, театра.
2) Заместо названия Break Room в медотделе теперь зона Med Restroom.
3) Заместо названия Hallway на саттелите теперь зона Ai satellite exterior hallway
4) Заменены большие фаерлоки у лестниц на их мелкий и более презентабельный аналог.
- Куча всего. Что не было лень - ушло к Пиро в ЛС.
- Нужна ещё пачка доработок.
Попозже изменю хирургичку робо для КПБ и КРБ
@Bizzonium Bizzonium self-requested a review as a code owner September 24, 2024 23:05
@Bizzonium Bizzonium merged commit a5b53dd into ss220-space:master220 Sep 25, 2024
12 checks passed
@Bizzonium Bizzonium changed the title New Station "Nova" add: New Station "Nova" Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DMI Affects DMI files javascript Pull requests that update Javascript code MAP Affects map TGUI Affects TGUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants