Simplify Asset Transfer and Mint Functions

[bitzoic]

Description

There are currently multiple functions for the same functionality:

Transfer:

• transfer()
• force_transfer_to_contract()
• transfer_to_address()

Mint:

• mint()
• mint_to()
• mint_to_contract()
• mint_to_address()

This be reduced to just transfer(), mint(), mint_to(). Specific functions for separately minting and transferring to contracts and addresses are legacy code left over from before the Identity type was introduced and now only adds additional complexity.

Changes:

• force_transfer_to_contract() is now private
• transfer_to_address() is now private
• mint_to_contract() removed
• mint_to_address() removed
• Removed newlines in inline docs for better forc doc compatibility
• Updated burn() inline docs

Breaking Changes:

• force_transfer_to_contract() is now private
• transfer_to_address() is now private
• mint_to_contract() removed
• mint_to_address() removed

Closes

• Closes Simplify asset transfer functions and make force transfer functions private #5773

Checklist

• I have linked to any relevant issues.
• I have commented my code, particularly in hard-to-understand areas.
• I have updated the documentation where relevant (API docs, the reference, and the Sway book).
  • If my change requires substantial documentation changes, I have requested support from the DevRel team
• I have added tests that prove my fix is effective or that my feature works.
• I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
• I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
• I have requested a review from the relevant team or maintainers.

[SwayStar123]

Isnt this forcing an additional match statement for some situations that might not need one? The previous functions could have been used for gas golfing where it is known whether the transfer is going to be to a contract or address beforehand