fa icon indicating copy to clipboard operation
fa copied to clipboard
verified publisher icon FAForever

Disable area reclaim commands

Open Garanas opened this issue 1 year ago • 6 comments

Description of the proposed changes

Disables the area reclaim command. Area ground attack command is polished in https://github.com/FAForever/fa/pull/6317

Garanas avatar Jun 25 '24 06:06 Garanas

Area attack has a consensus that it's a beneficial feature for both strategic depth and aesthetics, and it should be added.

lL1l1 avatar Jun 25 '24 08:06 lL1l1

I would leave the area attack as well. I haven't seen any pushback against that change. Could also revert the reclaim rate changes in this PR or should I make a separate PR for that?

Tagada14 avatar Jun 26 '24 12:06 Tagada14

Reverting pull requests is best done with the revert option if that is available, see also:

image

It would be a separate pull request. Relevant pull request to revert: https://github.com/FAForever/fa/pull/6253

Garanas avatar Jun 26 '24 12:06 Garanas

@lL1l1 / @Tagada14 processed the feedback

Garanas avatar Jul 01 '24 20:07 Garanas

@Garanas AFAIK it was decided to not release the Pulsar this patch either, are we moving that to a separate PR?

Tagada14 avatar Jul 03 '24 20:07 Tagada14

@tagada14 Here's the reasoning to not release Pulsar: Discord Otherwise it's ready to go. The textures are the highest priority (Progress: https://github.com/FAForever/fa/pull/6319). Shield effects would be nice to have. Further balance feedback seems unlikely because of how rare FAF develop games are, combined with the useful situations for it being uncommon.

lL1l1 avatar Jul 04 '24 07:07 lL1l1