Description

EmitApprovalEvent should emit the owner's address instead of the precompile address.

Otherwise, when transferFrom is called, the emitted event incorrectly shows the precompile address as the approver instead of the actual owner.

This change is verified in a separate end-to-end test (added in a #253), since it's more appropriate to test this behavior at the integration level. You can confirm the expected behavior in the following commit: 43d0962

Closes: #318

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

• tackled an existing issue or discussed with a team member
• left instructions on how to review the changes
• targeted the main branch

Reviewers Checklist

All items are required.
Please add a note if the item is not applicable
and please add your handle next to the items reviewed
if you only reviewed selected items.

I have...

• added a relevant changelog entry to the Unreleased section in CHANGELOG.md
• confirmed all author checklist items have been addressed
• confirmed that this PR does not change production code
• reviewed content
• tested instructions (if applicable)
• confirmed all CI checks have passed