Add order parameter to FormAlter attribute

@donquixote pointed out the missing paramter.

Shoudlnt the parameter be typed? Its new and seems to be an ?orderinterface

Seems straight forward, we had a brief conversation in slack about any questions haha

LGTM

Yeah, the original FormAlter issue was open and in progress before the order issue landed, so we forgot. Also mentioned at 📌 Rethink #[Hook] attribute inheritance Active

Agreed this is RTBC.

Bumping prio, tagging, and adding related

Very small nit. Otherwise, this looks ready.

dww → credited donquixote → .

Suggestion applied, pipeline is green again. I think we can sort out the public stuff at 📌 Rethink #[Hook] attribute inheritance Active

Crediting @donquixote for pointing out this omission (per comment #5), and MR and Slack reviews.

Generally I support this, thank you!

Committed 9d28549 and pushed to 11.x. Thanks!

I agree this could do with a bit more cleanup but let's defer that to 📌 Rethink #[Hook] attribute inheritance Active .

longwave → closed merge request !12067