Add cost event

[TateB]

Adds NameRegisteredWithCost and NameRenewedWithCost events

[Arachnid]

Why can't we just add this to the existing NameRegistered and NameRenewed events?

[TateB]

Why can't we just add this to the existing NameRegistered and NameRenewed events?

Those event handlers are for the BaseRegistrar events, which don't emit the cost values.

The options were to either:
a. Remove the BaseRegistrar event handlers and instead only have a handler for each controller's events
or
b. Keep the BaseRegistrar event handlers and create new handlers for each controller's events

I chose b as it adds the cost values while also maintaining forward compatibility with other controllers.

[Arachnid]

Can't we combine the two? The event doesn't have to be a 1:1 correspondence with a log; we can incorporate information from both events into one entity where available.