Skip to content

Port automatic jetpack activation - #88

Open
JustTNE wants to merge 2 commits into
DeltaV-Station:masterfrom
JustTNE:auto-jetpack
Open

Port automatic jetpack activation#88
JustTNE wants to merge 2 commits into
DeltaV-Station:masterfrom
JustTNE:auto-jetpack

Conversation

@JustTNE

@JustTNE JustTNE commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

About the PR

Port DeltaV-Station/Delta-v#4990

Why / Balance

See DeltaV-Station/Delta-v#4990

Technical details

See DeltaV-Station/Delta-v#4990

Test plan

Pick up jetpack, activate, observe automatic activation message. Walk off-grid. Observe automatic jetpack activation.

Requirements

  • I have tested this pull request and written instructions on how to test it
  • I have added media to this PR or it does not require an in-game showcase.

Licensing

Breaking changes

None

Comment thread Content.Shared/Movement/Systems/SharedJetpackSystem.cs Outdated
Comment thread Content.Shared/_DVA/Movement/Systems/SharedJetpackSystem.cs
Comment thread Content.Shared/_DVA/Movement/Systems/SharedJetpackSystem.cs
Comment on lines +7 to +12
public abstract partial class SharedJetpackSystem
{
[Dependency] private SharedTransformSystem _transform = default!;

private void OnJetpackToggle(Entity<JetpackComponent> jetpack, ref ToggleJetpackEvent args)
{

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this whole thing should be its own system

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't make this. This is a port. You'd need to pull in the original author @SirWarock if you want to have major changes :P

Comment thread Content.Shared/_DVA/Movement/Systems/SharedJetpackSystem.cs Outdated
Co-authored-by: pathetic meowmeow <uhhadd@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants