Removed partially custom BlockId implementation

[marioiordanov]

Usage related changes

Development related changes

Implemented as_ref method for wrapper BlockId(starknet-rs::BlockId) to return reference to its inner type.
Removed usage if any of our custom BlockId implementation and left only its custom deserialisation.
Refactored methods to receive &BlockId as an argument, instead of BlockId, to avoid copying of 32 bytes. Copying is done because starknet-rs BlockId implements the Copy trait.

Checklist:

• Applied formatting - ./scripts/format.sh
• No linter errors - ./scripts/clippy_check.sh
• No unused dependencies - ./scripts/check_unused_deps.sh
• Performed code self-review
• Rebased to the last commit of the target branch (or merged it into my branch)
• Documented the changes
• Linked the issues which this PR resolves
• Checked the TODO section in README.md if this PR resolves it
• Updated the tests
• All tests are passing - cargo test

[FabijanC]

Looks good, I assume the remaining checkboxes could be checked. Also, even though the PR title contains the word "partially", we don't plan to make more changes related to this, right?

[marioiordanov]

Looks good, I assume the remaining checkboxes could be checked. Also, even though the PR title contains the word "partially", we don't plan to make more changes related to this, right?

No, no more changes related to BlockId