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

ToolsForGateDungeon #2494

Merged
merged 12 commits into from
Feb 6, 2025
Merged

Conversation

21Melkuu
Copy link

@21Melkuu 21Melkuu commented Jan 17, 2025

Описание PR

Инструментарий для гейта из ыы13 и по совместительству пункт из цепочки РНД обновлений.
Нащиткодил полные штаны и доволен.

Компонента для создания отдельной мапы/данжа + логика для гейтгана.

!Это только инструментарий! Я не предоставляю карты самих гейтов, так как заниматься этим крайне в падлу. Если это апрувнут свяжемся с кем-то и обкашляем вопросики.
будущий гайтган, который должен выдаваться раундстартом, сам по себе сильное оружие, но в виду баланса им можно стрелять только в гейте.

До полной игры в гейт не хватает:
Прототип оружия гейтгана
Прототип роли работы
Самих карт для гейта
Возможно прототипы новых врагов, с новыми прототипами вооружения.

Технические детали
Для корректной работы нужно: GatewayDungeonStart на станции, GatewayDungeonMid - начала самого гейта, GatewayDungeonEnd - выходная точка из гейта, GatewayDungeonEndToStation - точка спавна после выхода из гейта.
Возможно стоило сделать более гибкую работу этого компонента, так например, если не будет всех 4 порталов (со станции -> в данж -> из данжа -> до нового портала(метки) на станцию) порталы не будут работать. Хотя если что-то в этой цепочке нет, наверное, не стоит пускать туда игроков.

За фильтрацию стрельбы гейтгана отвечает компонент GateDungeonMapComponent, его нужно будет добавить на все гриды, на которых должна быть разрешена стрельба

За загрузку мапы отвечает поле компонента GateDungeonComponent PathDungeon, который нужно закинуть в прототипе самих врат

Можно устанавливать несколько всех типы порталов.

Проверки

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

Изменения

🆑 Vimenant

  • add: Добавлены сломанные гейт-врата

@github-actions github-actions bot added Changes: Sprites Изменение затронуло файлы ".rsu / .png" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" labels Jan 17, 2025
Copy link

github-actions bot commented Jan 17, 2025

RSI Diff Bot; head commit 66424e5 merging into 69f8325
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Structures/Machines/gate_dungeon.rsi

State Old New Status
framebroken Added

Edit: diff updated after 66424e5

@Kirus59 Kirus59 self-assigned this Jan 23, 2025
@Kirus59 Kirus59 added the Need to be discussed Требуется обсуждение перед аппрувом/мёрджем label Jan 23, 2025
@Surani1
Copy link

Surani1 commented Jan 24, 2025

мелкуу показал дифф чеков по базе

@21Melkuu
Copy link
Author

21Melkuu commented Feb 4, 2025

@UrPrice

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 5, 2025

@UrPrice

Посмотрю сегодня

@21Melkuu 21Melkuu requested a review from Kirus59 February 5, 2025 11:30
Copy link

@Kirus59 Kirus59 left a comment

Choose a reason for hiding this comment

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

Уэээээ, в чеке походу спавнит инициализированную карту для проверки энтити.
Ну короче есть 2 стула - либо спавнить карту после зарядки гейта, либо удалять карту вместе с удалением гейта.

Ибо чек у тебя падает потому, что кол-во энтити перед спавном гейта и после его удаления не совпадает.

@21Melkuu
Copy link
Author

21Melkuu commented Feb 6, 2025

@Kirus59 чек побежден!

@Kirus59 Kirus59 removed Status: Needs Review Need to be discussed Требуется обсуждение перед аппрувом/мёрджем labels Feb 6, 2025
@Kirus59 Kirus59 merged commit 8861ab7 into SerbiaStrong-220:master Feb 6, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants