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.

Sign up for GitHub

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

Jump to bottom

Minor adjustment for D2k #20299

Merged
merged 1 commit into from Sep 21, 2022
Merged

Minor adjustment for D2k #20299

merged 1 commit into from Sep 21, 2022

Conversation

Porenutak
Copy link
Contributor

I propose these small changes that make gameplay easier in d2k

  • Add rally point for Palace: usefull for Saboteur and Fremen support powers. Downside: Rally point will be also available for harkonnen palace which has no use for it. (Adding RequiresCondition for Rallypoint is beyond my scope)
  • Remove harvester MustBeDestroyed trait for campaings: looking for every single harvester on fog of war maps can be annoying.
  • Allow Carryalls owned by player to be visible under the fog: so you know where your harvesters are going.
  • Increased CameraRemoveDelay on Superweapons. On default values view was removed before you can see impact.

@penev92 penev92 added this to the Next release milestone Sep 16, 2022
@Porenutak
Minor adjustment into the D2k
7301dde 
Minor adjustment into the D2k

- Add rally point into the Palace
- removed harvester MustBeDestroyed in campaing
- Players can see they carryalls and ornothopers under the fog.
- Increased CameraRemoveDelay on Superweapons so player can see superweapon impact.
@PunkPun
Copy link
Member

PunkPun commented Sep 17, 2022

I'm not sure about the harvester change, as I haven't played any D2K missions but otherwise this is adding some really nice polish

@Porenutak
Copy link
Contributor Author

Porenutak commented Sep 21, 2022 •
edited

I'm not sure about the harvester change, as I haven't played any D2K missions but otherwise this is adding some really nice polish

I go through all missions. No regression found

@abcdefg30 abcdefg30 merged commit ae7fc11 into OpenRA:bleed Sep 21, 2022
@abcdefg30
Copy link
Member

Changelog

@PunkPun PunkPun mentioned this pull request Oct 1, 2022
Closed
@Porenutak Porenutak deleted the Minor-adjustments-for-D2k branch November 14, 2023 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MustaphaTR MustaphaTR MustaphaTR left review comments

@Mailaender Mailaender Mailaender approved these changes

@abcdefg30 abcdefg30 abcdefg30 approved these changes

Assignees
No one assigned
Labels
PR: Needs +2
Projects
None yet
Milestone
Release 20230225
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@Porenutak @PunkPun @abcdefg30 @Mailaender @MustaphaTR @penev92