[Redelegation] fix: EventRedelegation unmarshaling failure

[red-0ne]

Summary

Unmarshalling transaction results logic in the delegation client is outdated, making the RelayMiner to panic when listening for these events.

This PR uses tx.UnmarshalTxResult from txClient to unmarshal the transaction.

Issue

When the RelayMiner is running and observing EventRedelegations by listening to application transactions, if a stake application occurs, the client tries to unmarshal it with a "pre-migration" code, causing the RelayMiner to panic

Type of change

Select one or more:

• New feature, functionality or library
• Bug fix
• Code health or cleanup
• Documentation
• Other (specify)

Testing

• Unit Tests: make go_develop_and_test
• LocalNet E2E Tests: make test_e2e
• DevNet E2E Tests: Add the devnet-test-e2e label to the PR. THIS IS VERY EXPENSIVE, so only do it after all the reviews are complete.
• Documentation changes: make docusaurus_start

Sanity Checklist

• I have tested my changes using the available tooling
• I have commented my code
• I have performed a self-review of my own code; both comments & source code
• I create and referenced any new tickets, if applicable
• I have left TODOs throughout the codebase, if applicable

[bryanchriswhite]

Nice one! 👌

IIRC correctly, when we were discussing this, we couldn't come up with a great reason not to export cometTxEvent. Otherwise this LGTM. 💪

[github-actions]

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks. If you just created a pull request, you might need to push another commit to produce a container image DevNet can utilize to spin up infrastructure. You can use make trigger_ci to push an empty commit.

[h5law]

LGTM Short and sweet nice changes :)