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 lavaland ruins #973

Merged
merged 34 commits into from
Feb 8, 2024
Merged

Add: New lavaland ruins #973

merged 34 commits into from
Feb 8, 2024

Conversation

Kar4es
Copy link

@Kar4es Kar4es commented Jan 30, 2024

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

Добавляет три руины в игру.

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

Много руин - интереснее игра.

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

image
image
image

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

заспавнил их и повзрывал, косяков вроде не нашел

Changelog

🆑
add: Добавлено три новых руины: Храм Чизуза, Заброшенный аванпост шахтеров и старый лагерь эшей.
/:cl:

@github-actions github-actions bot added the 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. label Jan 30, 2024
@github-actions github-actions bot added the 💾 Изменение конфига Ф-ф-фуриор...? label Jan 30, 2024
@konushi
Copy link

konushi commented Jan 30, 2024

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

@Kar4es
Copy link
Author

Kar4es commented Jan 31, 2024

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

я ничего не портировал, все оригинальное kekw

@konushi
Copy link

konushi commented Jan 31, 2024

я шиз, че то похожее видел значит...

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.

Небольшие изменения, ща карты пойду гляну

modular_ss220/maps220/code/RandomRuins/lavaland_ruins.dm Outdated Show resolved Hide resolved
modular_ss220/maps220/code/RandomRuins/lavaland_ruins.dm Outdated Show resolved Hide resolved
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.

Нет зоны
image

Поставь сундучки эшей, а то странно что просто на полу валяется, они как никак складируют
image

В целом, как-то пустовато у эшей, я понимаю что они съебали и забрали самое ценное, но нет бардака, накинь какого-то хлама, камней там, чисто декораций, и может каких-то безделушек как сказал кону

Далее...
Зоны, зона в руине должна покрывать всю руину, кроме пустоты естессна
image

Вот так
image

Все плитки пола не lavaland_air
Это значит что атмос будет активен если руина выпадет - не хорошо
Создай lavaland air аналог для травы, и используй lavaland air полы, с изменёнными стейтами, дерево если что есть лавалендское
image

Ну и побольше бардака бы, разрушений, как минимум пол слегка поломать, а то только пыль

@Kar4es
Copy link
Author

Kar4es commented Jan 31, 2024

Нет зоны image

Поставь сундучки эшей, а то странно что просто на полу валяется, они как никак складируют image

В целом, как-то пустовато у эшей, я понимаю что они съебали и забрали самое ценное, но нет бардака, накинь какого-то хлама, камней там, чисто декораций, и может каких-то безделушек как сказал кону

Далее... Зоны, зона в руине должна покрывать всю руину, кроме пустоты естессна image

Вот так image

Все плитки пола не lavaland_air Это значит что атмос будет активен если руина выпадет - не хорошо Создай lavaland air аналог для травы, и используй lavaland air полы, с изменёнными стейтами, дерево если что есть лавалендское image

Ну и побольше бардака бы, разрушений, как минимум пол слегка поломать, а то только пыль

на то и рассчет, что нет защиты от бури

@Kar4es
Copy link
Author

Kar4es commented Jan 31, 2024

на то и рассчет, что нет защиты от бури

у эшей

@AyIong
Copy link
Collaborator

AyIong commented Jan 31, 2024

на то и рассчет, что нет защиты от бури

Причём тут защита от бури?
Я про то что зоны нет
Глянь руины оффов какие там зоны снаружи и внутри руины

/area/lavaland/surface/outdoors - снаружи
Ну и внутри то что у тебя в чизусе, на оффах правда отдельная зона под каждую руину, наверное так и стоит делать, а не одну для всех, но да похуй

@Kar4es Kar4es requested a review from AyIong February 5, 2024 15:31
@AyIong
Copy link
Collaborator

AyIong commented Feb 7, 2024

В следующий раз сам откроешь, а то я драфты не смотрю

@AyIong AyIong marked this pull request as ready for review February 7, 2024 06:47
AyIong
AyIong previously approved these changes Feb 7, 2024
Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Про турф хелперы не забыл?

modular_ss220/maps220/code/RandomRuins/lavaland_ruins.dm Outdated Show resolved Hide resolved
modular_ss220/maps220/code/RandomRuins/lavaland_ruins.dm Outdated Show resolved Hide resolved
Co-authored-by: dj-34 <[email protected]>
@Kar4es
Copy link
Author

Kar4es commented Feb 7, 2024

Про турф хелперы не забыл?

Вроде как нет, потом еще проверять буду

@Kar4es Kar4es requested review from AyIong and dj-34 February 7, 2024 13:52
modular_ss220/maps220/code/RandomRuins/lavaland_ruins.dm Outdated Show resolved Hide resolved
modular_ss220/maps220/code/RandomRuins/lavaland_ruins.dm Outdated Show resolved Hide resolved
description = "Старый заброшенный аванпост. Его постигла участь разлома лаваленда."
prefix = "_maps/map_files220/RandomRuins/LavaRuins/"
suffix = "old_avanpost.dmm"
cost = 15 // на базе в сейфе есть ЕКА
Copy link
Collaborator

Choose a reason for hiding this comment

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

Не сильно ЕКА? (уточните кто-нибудь, а?)

Copy link
Author

Choose a reason for hiding this comment

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

обсуждал с Эйлонгом, он просил запрятать получше. Насчет стоимости не уверен

Co-authored-by: dj-34 <[email protected]>
Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Без кринжа
old_avanpost.dmm -> old_outpost.dmm

@Kar4es
Copy link
Author

Kar4es commented Feb 7, 2024

Без кринжа old_avanpost.dmm -> old_outpost.dmm

бля

Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Насчет ЕКА хз, в остальном вроде окей?

@AyIong
Copy link
Collaborator

AyIong commented Feb 7, 2024

А ЕКА откуда? Не было же

@AyIong AyIong dismissed their stale review February 7, 2024 20:09

Ека которого не было

@dj-34 dj-34 added the 🕛 Ждёт мержа Совсем скоро это будет залито label Feb 8, 2024
@Legendaxe Legendaxe merged commit cb5f8c9 into ss220club:master Feb 8, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕛 Ждёт мержа Совсем скоро это будет залито 💾 Изменение конфига Ф-ф-фуриор...? 📜 CL валиден 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants