Inconsistent transactionId field name on receipts/events

[peterbroadhurst]

We have a mix of:

• transactionHash
• transactionID
• transactionId

Receipts seem to be the place that should change:

firefly-fabconnect/internal/messages/messages.go

Lines 140 to 155 in 9b005b6

	// TransactionReceipt is sent when a transaction has been successfully mined
	type TransactionReceipt struct {
	ReplyCommon
	BlockNumber uint64 `json:"blockNumber"`
	SignerMSP string `json:"signerMSP"`
	Signer string `json:"signer"`
	TransactionID string `json:"transactionID"`
	Status string `json:"status"`
	}
	type ErrorReply struct {
	ReplyCommon
	ErrorMessage string `json:"errorMessage,omitempty"`
	OriginalMessage string `json:"requestPayload,omitempty"`
	TXHash string `json:"transactionHash,omitempty"`
	}

To match events which use transactionId:

firefly-fabconnect/internal/events/api/event.go

Lines 64 to 73 in afde10c

	type EventEntry struct {
	ChaincodeId string `json:"chaincodeId"`
	BlockNumber uint64 `json:"blockNumber"`
	TransactionId string `json:"transactionId"`
	TransactionIndex int `json:"transactionIndex"`
	EventName string `json:"eventName"`
	Payload interface{} `json:"payload"`
	Timestamp int64 `json:"timestamp,omitempty"`
	SubID string `json:"subId"`
	}