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

Map: Ревамп Синдибазы #1495

Closed
wants to merge 16 commits into from

Conversation

VERG-SS220
Copy link

@VERG-SS220 VERG-SS220 commented Sep 11, 2024

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

Делает кучу интересных улучшений на синдибазе. А то чё этот кусок астероида себе позволяет?
TO-DO List:

  • Добавить возможность зайти за своего персонажа на базу.
  • Запихнуть как можно больше кода из этого ПРа в модуль (НЕ ЗАБЫТЬ!!!!)
  • Добавить принтер на Синдибазу для упрощения РП.

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

Пока на станции происходит экшен, кровь, кишки, расчлененка, чё госты делают? Правильно, ждут антажку. Потому что делать им больше в этой игре нехуй, чем сидеть АФК после смерти 2 часа.

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

Как было до:
image
Как стало после (WIP):
image

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

Всё работает стабильно, дверка работает, ящик открывается (пока он пустой.)

Changelog

🆑 V.E.R.G
add: Добавили принтер на Синдибазу. Теперь факсы вести проще.
wip: Плнируется добавить пару новых функций на Синдибазу
/:cl:

@Kar4es
Copy link

Kar4es commented Sep 11, 2024

Ты даже если планируешь сделать тайпан MK2, то сделай его через модуль. В лист добавь это.

Так же не забывай что поддерживать его будешь только ты

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.

Сразу хочу предупредить, что если ты не будешь поддерживать обновами (хотя бы копипиздингом того, что делается у оффов на их синдибазе), то это сразу отклонение. Вряд ли кому-то будет интересно поддерживать то что было сделано другим, ну и соответственно зачем тогда оно нужно, когда на оффах занимаются обновлением.

@m-dzianishchyts
Copy link
Collaborator

За последние 3 года синди база обновилась ровно 1 раз, пути правятся скриптом

VERG-SS220 and others added 5 commits September 12, 2024 17:16
Ничего путного пока нет, думаем пока над тем, как переделать респавн

Co-Authored-By: Mikhail Dzianishchyts <[email protected]>
@github-actions github-actions bot added the 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. label Sep 14, 2024
Надо будет ещё карту либо перекидывать в модуль, либо как-то менять хардкодом базу.
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

Нене, переводы в другую ветку, это откатывай

@VERG-SS220
Copy link
Author

Нене, переводы в другую ветку, это откатывай

Кстати, принтер-то я на карту засунул, а в игре его нет. Не из того места сосёт, получается?

Пока проходит планировка по РнД и возможным косметическим изменениям
Переложил плитку на пластитановую. В ящик нужно будет закинуть ресурсов на 5 или седьмые техи.
@VERG-SS220
Copy link
Author

Вполне вероятно, что придется менять конфиг. Ибо там один из чеков ругается, что мол в конфиге есть мапа, а в итоге она не используется

чтоб не срал больше ошибками
@github-actions github-actions bot added the 💾 Изменение конфига Ф-ф-фуриор...? label Sep 15, 2024
VERG-SS220 added a commit to VERG-SS220/Paradise-SS220 that referenced this pull request Sep 16, 2024
@VERG-SS220 VERG-SS220 marked this pull request as ready for review September 16, 2024 04:41
m-dzianishchyts pushed a commit that referenced this pull request Sep 16, 2024
Для ПРа #1495

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

Тупо закидывает оффовскую синдибазу в нашу папку. Нужно для просмотра
диффа в ПРе #1495

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

<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

## Изображения изменений
До всех махинаций:


![image](https://github.com/user-attachments/assets/316b03b5-6572-46d9-b263-1ea780d2d164)
@github-actions github-actions bot added the Merge Conflict Ну блять... label Sep 16, 2024
@m-dzianishchyts m-dzianishchyts marked this pull request as draft September 16, 2024 11:58
@VERG-SS220
Copy link
Author

Бля, 305 ебучих конфликтов...

Этот ПР будет страдать вечно :jokerge:

@VERG-SS220 VERG-SS220 marked this pull request as ready for review September 17, 2024 14:40
@VERG-SS220 VERG-SS220 marked this pull request as draft September 17, 2024 14:42
Надеюсь, это все блять...
Следующим коммитом верну свою
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Sep 17, 2024
Дифф есть итак уже, не трогайте плиз
@github-actions github-actions bot added the Merge Conflict Ну блять... label Sep 17, 2024
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Sep 17, 2024
Бля, как так вообще случилось :xdd:
@m-dzianishchyts
Copy link
Collaborator

Не нравится пол. Хочешь сделать красиво - покрась лучше тайлы красным вдоль стен. Расставь декали где-то. А лучше перечитать Синдибаза. Сбор Фидбека и сделать сначала то, что там попросили

@VERG-SS220 VERG-SS220 closed this Sep 17, 2024
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.

5 participants