Skip to content

Comments

Fixed the issue that the move mission of the jumpjet does not end correctly#2100

Merged
TaranDahl merged 2 commits intoPhobos-developers:developfrom
TaranDahl:MissionMoveFix
Feb 23, 2026
Merged

Fixed the issue that the move mission of the jumpjet does not end correctly#2100
TaranDahl merged 2 commits intoPhobos-developers:developfrom
TaranDahl:MissionMoveFix

Conversation

@TaranDahl
Copy link
Contributor

Which would prevent AttackMove from finishing.

@TaranDahl TaranDahl added ❓Vanilla bug Vanilla game bugs that are requested to be fixed Needs testing ⚙️T1 T1 maintainer review is sufficient Bugfix This is a bugfix that does not need documentation beyond mention in changelog labels Feb 11, 2026
@github-actions
Copy link

github-actions bot commented Feb 11, 2026

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@Coronia
Copy link
Contributor

Coronia commented Feb 11, 2026

what's the difference between this and #2084?

@TaranDahl
Copy link
Contributor Author

what's the difference between this and #2084?

From a gameplay perspective, when the jumpjet uses AttackMove with a cell as the target, 2100 addresses the issue that the AttackMove task will not be cleared even after reaching the destination, and 2084 addresses the issue that the unit will not correctly switch to the attack task after acquiring the target.

@TaranDahl
Copy link
Contributor Author

Tested by
image

@TaranDahl TaranDahl added Tested Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions. and removed Needs testing labels Feb 22, 2026
Update Hooks.BugFixes.cpp

docs
@TaranDahl TaranDahl merged commit 819afa2 into Phobos-developers:develop Feb 23, 2026
6 checks passed
@TaranDahl TaranDahl deleted the MissionMoveFix branch February 23, 2026 17:30
DeathFishAtEase added a commit to DeathFishAtEase/Phobos that referenced this pull request Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugfix This is a bugfix that does not need documentation beyond mention in changelog ⚙️T1 T1 maintainer review is sufficient Tested ❓Vanilla bug Vanilla game bugs that are requested to be fixed Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants