fix!: clear expiration queues on revoke

[beer-1]

Description

When removing an entry from the expiration queue, x/feegrant uses the key with the granter/grantee inverted. The correct key is feegrant.FeeAllowanceKey(granter, grantee), not feegrant.FeeAllowanceKey(grantee, granter). This prevents proper deletion of the allowance entry.

Patch-style hint

- key := feegrant.FeeAllowanceKey(grantee, granter)
+ key := feegrant.FeeAllowanceKey(granter, grantee)

Impact

Expiration not actually removed for the prior grant.
Recreated grant can be deleted later by the leftover (stale) expiration event.

!!!!!! consensus breaking !!!!!!

[aljo242]

can you target this PR against main and backport from there?

[beer-1]

@aljo242 but there is no problem in main branch after updating x/feegrant to use collection

[aljo242]

@beer-1 can we document how this is state breaking? and how folks who have x/feegrant integrated should go about upgrading

[beer-1]

@beer-1 can we document how this is state breaking? and how folks who have x/feegrant integrated should go about upgrading

Yea sure. Can you let me know where should I put that docs part? Changelog?
for now, added in changelog