Modify function visibilities in Coordinator contract

[abandeali1]

Description

The following methods are now public:

• decodeOrdersFromFillData
• getCoordinatorApprovalHash
• getTransactionHash

assertValidTransactionOrdersApproval is now internal. It didn't make sense for this to be public given that the passed in orders can differ from the orders in transaction.data, potentially leading to incorrect validation. It is always better to use assertValidCoordinatorApprovals.

Testing instructions

Types of changes

Checklist:

• Prefix PR title with [WIP] if necessary.
• Add tests to cover changes as needed.
• Update documentation as needed.
• Add new entries to the relevant CHANGELOG.jsons.

[coveralls]

Coverage remained the same at 85.174% when pulling 17ff262 on feat/coordinator/functionVisibility into 4f25ff6 on development.

[dorothy-zbornak]

👀 👍