write_acknowledgement event doesn't contain channel ordering

[benluelo]

Summary of Bug

ibc-go/modules/core/04-channel/keeper/events.go

Lines 177 to 200 in 9bccf22

	func emitWriteAcknowledgementEvent(ctx sdk.Context, packet types.Packet, channel types.Channel, acknowledgement []byte) {
	ctx.EventManager().EmitEvents(sdk.Events{
	sdk.NewEvent(
	types.EventTypeWriteAck,
	sdk.NewAttribute(types.AttributeKeyDataHex, hex.EncodeToString(packet.GetData())),
	sdk.NewAttribute(types.AttributeKeyTimeoutHeight, packet.GetTimeoutHeight().String()),
	sdk.NewAttribute(types.AttributeKeyTimeoutTimestamp, fmt.Sprintf("%d", packet.GetTimeoutTimestamp())),
	sdk.NewAttribute(types.AttributeKeySequence, fmt.Sprintf("%d", packet.GetSequence())),
	sdk.NewAttribute(types.AttributeKeySrcPort, packet.GetSourcePort()),
	sdk.NewAttribute(types.AttributeKeySrcChannel, packet.GetSourceChannel()),
	sdk.NewAttribute(types.AttributeKeyDstPort, packet.GetDestPort()),
	sdk.NewAttribute(types.AttributeKeyDstChannel, packet.GetDestChannel()),
	sdk.NewAttribute(types.AttributeKeyAckHex, hex.EncodeToString(acknowledgement)),
	// we only support 1-hop packets now, and that is the most important hop for a relayer
	// (is it going to a chain I am connected to)
	sdk.NewAttribute(types.AttributeKeyConnection, channel.ConnectionHops[0]), // DEPRECATED
	sdk.NewAttribute(types.AttributeKeyConnectionID, channel.ConnectionHops[0]),
	),
	sdk.NewEvent(
	sdk.EventTypeMessage,
	sdk.NewAttribute(sdk.AttributeKeyModule, types.AttributeValueCategory),
	),
	})
	}

ibc-go/docs/events/events.md

Line 235 in 9bccf22

| write_acknowledgement | packet_channel_ordering | \{channel.Ordering\} | |

the doc is inconsistent with the implementation here; the write ack event doesn't actually contain the ordering - it is notably the only packet event that doesn't have that field

Expected Behaviour

write_acknowledgement contains a channel ordering field.

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned